*: tidy up usage of char **environ

diff --git a/archival/libunarchive/open_transformer.c b/archival/libunarchive/open_transformer.c
index 757a2a38905e2618309a28854cc26c6789c6f107..d6f5e62717caba6fd4339eeca71529c89488d1e2 100644 (file)
--- a/archival/libunarchive/open_transformer.c
+++ b/archival/libunarchive/open_transformer.c
@@ -30,7 +30,7 @@ int open_transformer(int src_fd,
 
        if (pid == 0) {
                /* child process */
-               close(fd_pipe[0]); /* We don't wan't to read from the parent */
+               close(fd_pipe[0]); /* We don't want to read from the parent */
                // FIXME: error check?
 #if BB_MMU
                transformer(src_fd, fd_pipe[1]);

diff --git a/coreutils/env.c b/coreutils/env.c
index b45cbcc7dc2f6b1eb817dcaf17e0de351e08ed87..7273d39fc9f1a533cf295b34273a5ece26242bee 100644 (file)
--- a/coreutils/env.c
+++ b/coreutils/env.c
@@ -30,8 +30,6 @@
  */
 
 #include <getopt.h> /* struct option */
-extern char **environ;
-
 #include "libbb.h"
 
 #if ENABLE_FEATURE_ENV_LONG_OPTIONS

diff --git a/coreutils/printenv.c b/coreutils/printenv.c
index bac77e2a8e287aa2e9e24cb04907d3e1a6347492..06d68d0ef48693782b7f741942d4b87da19d62b5 100644 (file)
--- a/coreutils/printenv.c
+++ b/coreutils/printenv.c
@@ -9,7 +9,6 @@
  */
 
 #include "libbb.h"
-extern char **environ;
 
 int printenv_main(int argc, char **argv) MAIN_EXTERNALLY_VISIBLE;
 int printenv_main(int argc, char **argv)

diff --git a/editors/awk.c b/editors/awk.c
index da3401b377817e12b151b08a2b08b4cd2f58aab2..983b3111642f3d1684746c17d4eb994809d4e1fa 100644 (file)
--- a/editors/awk.c
+++ b/editors/awk.c
@@ -10,7 +10,6 @@
 #include "libbb.h"
 #include "xregex.h"
 #include <math.h>
-extern char **environ;
 
 /* This is a NOEXEC applet. Be very careful! */
 

diff --git a/include/libbb.h b/include/libbb.h
index 9d70e70995903b3282a6d251375dd5c64f8173a1..c5b685985e91dd504a60a2159631cca0bfe22d57 100644 (file)
--- a/include/libbb.h
+++ b/include/libbb.h
@@ -67,6 +67,9 @@
 #include "shadow_.h"
 #endif
 
+/* Some libc's don't declare it, help them */
+extern char **environ;
+
 #if defined(__GLIBC__) && __GLIBC__ < 2
 int vdprintf(int d, const char *format, va_list ap);
 #endif

diff --git a/networking/inetd.c b/networking/inetd.c
index d643dc6e04651342f8879eb4b5ae53205576bd53..a7259f3d43ce084a2c80702bccaf84a5056bb6a6 100644 (file)
--- a/networking/inetd.c
+++ b/networking/inetd.c
@@ -172,9 +172,6 @@
 #include <rpc/pmap_clnt.h>
 #endif
 
-extern char **environ;
-
-
 #define _PATH_INETDPID  "/var/run/inetd.pid"
 
 #define CNT_INTVL       60              /* servers in CNT_INTVL sec. */

diff --git a/runit/runsv.c b/runit/runsv.c
index 84f5193f5792b0faddd82c5a812477e8e7faecb7..e9a0745802f44f5fd62d00d18d45b3fb00c573c7 100644 (file)
--- a/runit/runsv.c
+++ b/runit/runsv.c
@@ -279,7 +279,7 @@ static unsigned custom(struct svdir *s, char c)
                                        warn_cannot("setup stdout for control/?");
                                prog[0] = a;
                                prog[1] = NULL;
-                               execve(a, prog, environ);
+                               execv(a, prog);
                                fatal_cannot("run control/?");
                        }
                        while (safe_waitpid(pid, &w, 0) == -1) {

diff --git a/runit/svlogd.c b/runit/svlogd.c
index 1d679c9727ebaa85e86622e3149c675299f5b28f..9c169da1f3b9c3fe6e5738611a3061b72ea1649a 100644 (file)
--- a/runit/svlogd.c
+++ b/runit/svlogd.c
@@ -252,7 +252,7 @@ static unsigned processorstart(struct logdir *ld)
                prog[1] = (char*)"-c";
                prog[2] = ld->processor;
                prog[3] = NULL;
-               execve("/bin/sh", prog, environ);
+               execv("/bin/sh", prog);
                bb_perror_msg_and_die(FATAL"cannot %s processor %s", "run", ld->name);
        }
        ld->ppid = pid;

diff --git a/shell/ash.c b/shell/ash.c
index 96563bf06edc35218407a060e0016903f5bf3f79..2a9e96a3570c69061d717ae8b146f29b86a2a526 100644 (file)
--- a/shell/ash.c
+++ b/shell/ash.c
@@ -60,7 +60,6 @@
 #if JOBS || ENABLE_ASH_READ_NCHARS
 #include <termios.h>
 #endif
-extern char **environ;
 
 #if defined(__uClinux__)
 #error "Do not even bother, ash will not run on uClinux"

diff --git a/shell/hush.c b/shell/hush.c
index 9dc85d0bad1debb17b4b9c8a04abea36e4e24c2d..a75407634d4f87fa944cd82d418479dd9107b22e 100644 (file)
--- a/shell/hush.c
+++ b/shell/hush.c
@@ -81,8 +81,6 @@
 #include <getopt.h>    /* should be pretty obvious */
 /* #include <dmalloc.h> */
 
-extern char **environ; /* This is in <unistd.h>, but protected with __USE_GNU */
-
 #include "busybox.h" /* for APPLET_IS_NOFORK/NOEXEC */
 
 

diff --git a/shell/msh.c b/shell/msh.c
index 9e9b798a1771d1b8576158f7bcdfbf92e794e158..7371120cab73acc06fac65eabc547b3648981343 100644 (file)
--- a/shell/msh.c
+++ b/shell/msh.c
@@ -84,7 +84,6 @@ static char *itoa(int n)
 }
 #else
 # include "busybox.h" /* for applet_names */
-extern char **environ;
 #endif
 
 /*#define MSHDEBUG 1*/
@@ -2825,11 +2824,13 @@ static int forkexec(struct op *t, int *pin, int *pout, int act, char **wp)
 
        if (pin != NULL) {
                xmove_fd(pin[0], 0);
-               if (pin[1] != 0) close(pin[1]);
+               if (pin[1] != 0)
+                       close(pin[1]);
        }
        if (pout != NULL) {
                xmove_fd(pout[1], 1);
-               if (pout[1] != 1) close(pout[0]);
+               if (pout[1] != 1)
+                       close(pout[0]);
        }
 
        iopp = t->ioact;
@@ -4181,7 +4182,8 @@ static int grave(int quoted)
         *  echo "$files" >zz
         */
        xmove_fd(pf[1], 1);
-       if (pf[0] != 1) close(pf[0]);
+       if (pf[0] != 1)
+               close(pf[0]);
 
        argument_list[0] = (char *) DEFAULT_SHELL;
        argument_list[1] = (char *) "-c";