Fix invalid pointer arithmetic.

[nmh] / sbr / getarguments.c

diff --git a/sbr/getarguments.c b/sbr/getarguments.c
index fc3d2c05bc0bb99ae329f50a1789e9137a79dbed..28cbe465e308ee825da8dc8e4c30f9c492b1b958 100644 (file)
--- a/sbr/getarguments.c
+++ b/sbr/getarguments.c
@@ -1,8 +1,4 @@
-
-/*
- * getarguments.c -- Get the argument vector ready to go.
- *
- * $Id$
+/* getarguments.c -- Get the argument vector ready to go.
  *
  * This code is Copyright (c) 2002, by the authors of nmh.  See the
  * COPYRIGHT file in the root directory of the nmh distribution for
@@ -15,14 +11,14 @@
 char **
 getarguments (char *invo_name, int argc, char **argv, int check_context)
 {
-    char *cp, **ap, **bp, **arguments;
+    char *cp = NULL, **ap = NULL, **bp = NULL, **arguments = NULL;
     int n = 0;
 
     /*
      * Check if profile/context specifies any arguments
      */
     if (check_context && (cp = context_find (invo_name))) {
-       cp = getcpy (cp);               /* make copy    */
+       cp = mh_xstrdup(cp);            /* make copy    */
        ap = brkstring (cp, " ", "\n"); /* split string */
 
        /* Count number of arguments split */
@@ -31,11 +27,11 @@ getarguments (char *invo_name, int argc, char **argv, int check_context)
            n++;
     }
 
-    arguments = (char **) mh_xmalloc ((argc + n) * sizeof(*arguments));
+    arguments = mh_xmalloc ((argc + n) * sizeof(*arguments));
     bp = arguments;
 
     /* Copy any arguments from profile/context */
-    if (n > 0) {
+    if (ap != NULL && n > 0) {
        while (*ap)
            *bp++ = *ap++;
      }