- In main.c use NULL instead of 0
authorMax Okumoto <okumoto@dragonflybsd.org>
Fri, 4 Feb 2005 22:35:42 +0000 (22:35 +0000)
committerMax Okumoto <okumoto@dragonflybsd.org>
Fri, 4 Feb 2005 22:35:42 +0000 (22:35 +0000)
- In str.c check if brk_string() is passed a null pointer in
  store_argc.  If so then don't return argc.

Taken-from: FreeBSD

usr.bin/make/main.c
usr.bin/make/str.c
usr.bin/make/suff.c

index 7401f1a..d29d6b8 100644 (file)
@@ -38,7 +38,7 @@
  * @(#) Copyright (c) 1988, 1989, 1990, 1993 The Regents of the University of California.  All rights reserved.
  * @(#)main.c  8.3 (Berkeley) 3/19/94
  * $FreeBSD: src/usr.bin/make/main.c,v 1.35.2.10 2003/12/16 08:34:11 des Exp $
- * $DragonFly: src/usr.bin/make/main.c,v 1.53 2005/02/03 23:41:05 okumoto Exp $
+ * $DragonFly: src/usr.bin/make/main.c,v 1.54 2005/02/04 22:35:42 okumoto Exp $
  */
 
 /*-
@@ -407,7 +407,7 @@ chdir_verify_path(const char *path, char *obpath)
        if (stat(path, &sb) == 0 && S_ISDIR(sb.st_mode)) {
                if (chdir(path) == -1 || getcwd(obpath, MAXPATHLEN) == NULL) {
                        warn("warning: %s", path);
-                       return (0);
+                       return (NULL);
                }
                return (obpath);
        }
@@ -476,6 +476,7 @@ main(int argc, char **argv)
        const char *machine_arch = getenv("MACHINE_ARCH");
        const char *machine_cpu = getenv("MACHINE_CPU");
        char *cp = NULL, *start;
+
                                        /* avoid faults on read-only strings */
        static char syspath[] = _PATH_DEFSYSPATH;
 
@@ -645,6 +646,7 @@ main(int argc, char **argv)
 
        {
        struct stat sa;
+
        if (stat(curdir, &sa) == -1)
            err(2, "%s", curdir);
        }
index c217050..88947c8 100644 (file)
@@ -37,7 +37,7 @@
  *
  * @(#)str.c   5.8 (Berkeley) 6/1/90
  * $FreeBSD: src/usr.bin/make/str.c,v 1.12.2.2 2004/02/23 12:10:57 ru Exp $
- * $DragonFly: src/usr.bin/make/str.c,v 1.22 2005/01/27 02:30:19 okumoto Exp $
+ * $DragonFly: src/usr.bin/make/str.c,v 1.23 2005/02/04 22:35:42 okumoto Exp $
  */
 
 #include <ctype.h>
@@ -225,7 +225,8 @@ brk_string(char *str, int *store_argc, Boolean expand)
                *t++ = (char)ch;
        }
 done:  argv[argc] = NULL;
-       *store_argc = argc;
+       if ((store_argc != NULL)
+               *store_argc = argc;
        return (argv);
 }
 
index 745be2c..ebc9edc 100644 (file)
@@ -37,7 +37,7 @@
  *
  * @(#)suff.c  8.4 (Berkeley) 3/21/94
  * $FreeBSD: src/usr.bin/make/suff.c,v 1.12.2.2 2004/06/10 13:07:53 ru Exp $
- * $DragonFly: src/usr.bin/make/suff.c,v 1.31 2005/02/01 22:05:36 okumoto Exp $
+ * $DragonFly: src/usr.bin/make/suff.c,v 1.32 2005/02/04 22:35:42 okumoto Exp $
  */
 
 /*-
@@ -1618,7 +1618,8 @@ SuffFindArchiveDeps(GNode *gn, Lst *slst)
     char       *eoarch;    /* End of archive portion */
     char       *eoname;    /* End of member portion */
     GNode      *mem;       /* Node for member */
-    static const char  *copy[] = { /* Variables to be copied from the member node */
+    /* Variables to be copied from the member node */
+    static const char  *copy[] = {
        TARGET,             /* Must be first */
        PREFIX,             /* Must be second */
     };
@@ -2349,9 +2350,9 @@ Suff_PrintAll(void)
 #ifdef DEBUG_SRC
 /*
  * Printaddr --
- *     Print the address of a node, used as an interative function.
+ *     Print the address of a node.
  */
-int
+static int
 PrintAddr(void *a, void *b __unused)
 {
     printf("%p ", a);