new: Improve the fix with the NULL-sentinel.

With multiple sequence arguments on the command line you might end up
with garbage in the sequence[] array (it needs the last argument to be
a NULL. So do two things: make the first two arguments NULL, and as we
add arguments to sequence[] make sure the list has a NULL on the end.
This is another problem discovered by the nmh test suite.

Pulled in from nmh. Thanks to Ken Hornstein.

diff --git a/uip/new.c b/uip/new.c
index f6ac3642f95f2799d52481ec7c3e07d51a6185f9..dab17adab4d63ab81e71b29d9d906b8b230e5bea 100644 (file)
--- a/uip/new.c
+++ b/uip/new.c
@@ -433,6 +433,7 @@ main(int argc, char **argv)
        struct node *folder;
 
        sequences[0] = NULL;
+       sequences[1] = NULL;
 
 #ifdef LOCALE
        setlocale(LC_ALL, "");
@@ -483,6 +484,7 @@ main(int argc, char **argv)
                /* have a sequence argument */
                if (!seq_in_list(cp, sequences)) {
                        sequences[i++] = cp;
+                       sequences[i] = NULL;
                }
        }