init: fix logically inverted FEATURE_EXTRA_QUIET check
[oweals/busybox.git] / libbb / vfork_daemon_rexec.c
index 2c4c930b2bc3d5fe828464100525465e9159d11a..9be90172290ef67bfd611c64371befa24c2e56b8 100644 (file)
  * Licensed under GPLv2 or later, see file LICENSE in this tarball for details.
  */
 
-#include <paths.h>
 #include "busybox.h" /* uses applet tables */
 
 /* This does a fork/exec in one call, using vfork().  Returns PID of new child,
  * -1 for failure.  Runs argv[0], searching path if that has no / in it. */
-pid_t spawn(char **argv)
+pid_t FAST_FUNC spawn(char **argv)
 {
        /* Compiler should not optimize stores here */
        volatile int failed;
        pid_t pid;
 
-// Ain't it a good place to fflush(NULL)?
+       fflush_all();
 
        /* Be nice to nommu machines. */
        failed = 0;
@@ -42,6 +41,8 @@ pid_t spawn(char **argv)
                 * (but don't run atexit() stuff, which would screw up parent.)
                 */
                failed = errno;
+               /* mount, for example, does not want the message */
+               /*bb_perror_msg("can't execute '%s'", argv[0]);*/
                _exit(111);
        }
        /* parent */
@@ -58,7 +59,7 @@ pid_t spawn(char **argv)
 }
 
 /* Die with an error message if we can't spawn a child process. */
-pid_t xspawn(char **argv)
+pid_t FAST_FUNC xspawn(char **argv)
 {
        pid_t pid = spawn(argv);
        if (pid < 0)
@@ -66,9 +67,9 @@ pid_t xspawn(char **argv)
        return pid;
 }
 
-int safe_waitpid(int pid, int *wstat, int options)
+pid_t FAST_FUNC safe_waitpid(pid_t pid, int *wstat, int options)
 {
-       int r;
+       pid_t r;
 
        do
                r = waitpid(pid, wstat, options);
@@ -76,13 +77,13 @@ int safe_waitpid(int pid, int *wstat, int options)
        return r;
 }
 
-int wait_any_nohang(int *wstat)
+pid_t FAST_FUNC wait_any_nohang(int *wstat)
 {
        return safe_waitpid(-1, wstat, WNOHANG);
 }
 
 // Wait for the specified child PID to exit, returning child's error return.
-int wait4pid(int pid)
+int FAST_FUNC wait4pid(pid_t pid)
 {
        int status;
 
@@ -101,7 +102,7 @@ int wait4pid(int pid)
 }
 
 #if ENABLE_FEATURE_PREFER_APPLETS
-void save_nofork_data(struct nofork_save_area *save)
+void FAST_FUNC save_nofork_data(struct nofork_save_area *save)
 {
        memcpy(&save->die_jmp, &die_jmp, sizeof(die_jmp));
        save->applet_name = applet_name;
@@ -111,7 +112,7 @@ void save_nofork_data(struct nofork_save_area *save)
        save->saved = 1;
 }
 
-void restore_nofork_data(struct nofork_save_area *save)
+void FAST_FUNC restore_nofork_data(struct nofork_save_area *save)
 {
        memcpy(&die_jmp, &save->die_jmp, sizeof(die_jmp));
        applet_name = save->applet_name;
@@ -120,11 +121,12 @@ void restore_nofork_data(struct nofork_save_area *save)
        die_sleep = save->die_sleep;
 }
 
-int run_nofork_applet_prime(struct nofork_save_area *old, int applet_no, char **argv)
+int FAST_FUNC run_nofork_applet_prime(struct nofork_save_area *old, int applet_no, char **argv)
 {
        int rc, argc;
 
        applet_name = APPLET_NAME(applet_no);
+
        xfunc_error_retval = EXIT_FAILURE;
 
        /* Special flag for xfunc_die(). If xfunc will "die"
@@ -132,7 +134,30 @@ int run_nofork_applet_prime(struct nofork_save_area *old, int applet_no, char **
         * die_sleep and longjmp here instead. */
        die_sleep = -1;
 
-       /* option_mask32 = 0; - not needed */
+       /* In case getopt() or getopt32() was already called:
+        * reset the libc getopt() function, which keeps internal state.
+        *
+        * BSD-derived getopt() functions require that optind be set to 1 in
+        * order to reset getopt() state.  This used to be generally accepted
+        * way of resetting getopt().  However, glibc's getopt()
+        * has additional getopt() state beyond optind, and requires that
+        * optind be set to zero to reset its state.  So the unfortunate state of
+        * affairs is that BSD-derived versions of getopt() misbehave if
+        * optind is set to 0 in order to reset getopt(), and glibc's getopt()
+        * will core dump if optind is set 1 in order to reset getopt().
+        *
+        * More modern versions of BSD require that optreset be set to 1 in
+        * order to reset getopt().  Sigh.  Standards, anyone?
+        */
+#ifdef __GLIBC__
+       optind = 0;
+#else /* BSD style */
+       optind = 1;
+       /* optreset = 1; */
+#endif
+       /* optarg = NULL; opterr = 1; optopt = 63; - do we need this too? */
+       /* (values above are what they initialized to in glibc and uclibc) */
+       /* option_mask32 = 0; - not needed, no applet depends on it being 0 */
 
        argc = 1;
        while (argv[argc])
@@ -148,9 +173,9 @@ int run_nofork_applet_prime(struct nofork_save_area *old, int applet_no, char **
                rc = applet_main[applet_no](argc, tmp_argv);
 
        /* The whole reason behind nofork_save_area is that <applet>_main
-        * may exit non-locally! For example, in hush Ctrl-Z tries to
-        * (modulo bugs) to dynamically create child (backgrounded task)
-        * if it detects that Ctrl-Z was pressed when a NOFORK was running!
+        * may exit non-locally! For example, in hush Ctrl-Z tries
+        * (modulo bugs) to dynamically create child (backgrounded task)
+        * if it detects that Ctrl-Z was pressed when a NOFORK was running.
         * Testcase: interactive "rm -i".
         * Don't fool yourself into thinking "and <applet>_main() returns
         * quickly here" and removing "useless" nofork_save_area code. */
@@ -161,12 +186,20 @@ int run_nofork_applet_prime(struct nofork_save_area *old, int applet_no, char **
                        rc = 0;
        }
 
-       /* Restoring globals */
+       /* Restoring some globals */
        restore_nofork_data(old);
+
+       /* Other globals can be simply reset to defaults */
+#ifdef __GLIBC__
+       optind = 0;
+#else /* BSD style */
+       optind = 1;
+#endif
+
        return rc & 0xff; /* don't confuse people with "exitcodes" >255 */
 }
 
-int run_nofork_applet(int applet_no, char **argv)
+int FAST_FUNC run_nofork_applet(int applet_no, char **argv)
 {
        struct nofork_save_area old;
 
@@ -176,7 +209,7 @@ int run_nofork_applet(int applet_no, char **argv)
 }
 #endif /* FEATURE_PREFER_APPLETS */
 
-int spawn_and_wait(char **argv)
+int FAST_FUNC spawn_and_wait(char **argv)
 {
        int rc;
 #if ENABLE_FEATURE_PREFER_APPLETS
@@ -210,7 +243,7 @@ int spawn_and_wait(char **argv)
 }
 
 #if !BB_MMU
-void re_exec(char **argv)
+void FAST_FUNC re_exec(char **argv)
 {
        /* high-order bit of first char in argv[0] is a hidden
         * "we have (already) re-execed, don't do it again" flag */
@@ -219,40 +252,37 @@ void re_exec(char **argv)
        bb_perror_msg_and_die("exec %s", bb_busybox_exec_path);
 }
 
-void forkexit_or_rexec(char **argv)
+pid_t FAST_FUNC fork_or_rexec(char **argv)
 {
        pid_t pid;
        /* Maybe we are already re-execed and come here again? */
        if (re_execed)
-               return;
-
+               return 0;
        pid = vfork();
        if (pid < 0) /* wtf? */
                bb_perror_msg_and_die("vfork");
        if (pid) /* parent */
-               exit(0);
+               return pid;
        /* child - re-exec ourself */
        re_exec(argv);
 }
 #else
 /* Dance around (void)...*/
-#undef forkexit_or_rexec
-void forkexit_or_rexec(void)
+#undef fork_or_rexec
+pid_t FAST_FUNC fork_or_rexec(void)
 {
        pid_t pid;
        pid = fork();
        if (pid < 0) /* wtf? */
                bb_perror_msg_and_die("fork");
-       if (pid) /* parent */
-               exit(0);
-       /* child */
+       return pid;
 }
-#define forkexit_or_rexec(argv) forkexit_or_rexec()
+#define fork_or_rexec(argv) fork_or_rexec()
 #endif
 
 /* Due to a #define in libbb.h on MMU systems we actually have 1 argument -
  * char **argv "vanishes" */
-void bb_daemonize_or_rexec(int flags, char **argv)
+void FAST_FUNC bb_daemonize_or_rexec(int flags, char **argv)
 {
        int fd;
 
@@ -265,13 +295,21 @@ void bb_daemonize_or_rexec(int flags, char **argv)
                close(2);
        }
 
-       fd = xopen(bb_dev_null, O_RDWR);
+       fd = open(bb_dev_null, O_RDWR);
+       if (fd < 0) {
+               /* NB: we can be called as bb_sanitize_stdio() from init
+                * or mdev, and there /dev/null may legitimately not (yet) exist!
+                * Do not use xopen above, but obtain _ANY_ open descriptor,
+                * even bogus one as below. */
+               fd = xopen("/", O_RDONLY); /* don't believe this can fail */
+       }
 
        while ((unsigned)fd < 2)
                fd = dup(fd); /* have 0,1,2 open at least to /dev/null */
 
        if (!(flags & DAEMON_ONLY_SANITIZE)) {
-               forkexit_or_rexec(argv);
+               if (fork_or_rexec(argv))
+                       exit(EXIT_SUCCESS); /* parent */
                /* if daemonizing, make sure we detach from stdio & ctty */
                setsid();
                dup2(fd, 0);
@@ -286,7 +324,7 @@ void bb_daemonize_or_rexec(int flags, char **argv)
        }
 }
 
-void bb_sanitize_stdio(void)
+void FAST_FUNC bb_sanitize_stdio(void)
 {
        bb_daemonize_or_rexec(DAEMON_ONLY_SANITIZE, NULL);
 }