Another problem discovered by the test suite.

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.

diff --git a/uip/new.c b/uip/new.c
index 752f02831b997bfba91a96502ae681d5ceb86c70..38c8f0b0329799f8642597104217b145e618ae24 100644 (file)
--- a/uip/new.c
+++ b/uip/new.c
@@ -409,6 +409,7 @@ main(int argc, char **argv)
     struct node *folder;
 
     sequences[0] = NULL;
     struct node *folder;
 
     sequences[0] = NULL;

+    sequences[1] = NULL;

 
 #ifdef LOCALE
     setlocale(LC_ALL, "");
 
 #ifdef LOCALE
     setlocale(LC_ALL, "");


@@ -456,6 +457,7 @@ main(int argc, char **argv)
        /* have a sequence argument */
        if (!seq_in_list(cp, sequences)) {
            sequences[i++] = cp;
        /* have a sequence argument */
        if (!seq_in_list(cp, sequences)) {
            sequences[i++] = cp;

+           sequences[i] = NULL;

        }
     }
 
        }
     }