libbb: introduce and use BB_EXECVP_or_die()

function                                             old     new   delta
BB_EXECVP_or_die                                       -      47     +47
time_main                                           1042    1043      +1
chrt_main                                            371     364      -7
ionice_main                                          292     282     -10
setsid_main                                           69      56     -13
nohup_main                                           236     223     -13
cttyhack_main                                        266     253     -13
chroot_main                                           94      81     -13
chpst_main                                           746     733     -13
timeout_main                                         297     279     -18
taskset_main                                         541     522     -19
vfork_child                                           67      45     -22
parse                                                975     953     -22
lpd_main                                             770     748     -22
launch_helper                                        192     170     -22
tcpudpsvd_main                                      1810    1782     -28
nice_main                                            190     156     -34
env_main                                             242     206     -36
run_command                                          221     174     -47
------------------------------------------------------------------------------
(add/remove: 1/0 grow/shrink: 1/17 up/down: 48/-352)         Total: -304 bytes

Signed-off-by: Pascal Bellard <pascal.bellard@ads-lu.com>
Signed-off-by: Denys Vlasenko <vda.linux@googlemail.com>

diff --git a/console-tools/openvt.c b/console-tools/openvt.c
index 6f58916e7b61c5b3b16b4ff60c91b14d53e71f64..e3ea71bc56211f5f90daff532e79086a96228b4b 100644 (file)
--- a/console-tools/openvt.c
+++ b/console-tools/openvt.c
@@ -97,8 +97,7 @@ static NOINLINE void vfork_child(char **argv)
                //bb_error_msg("our pgrp %d", getpgrp());
                //bb_error_msg("VT's sid %d", tcgetsid(0));
                //bb_error_msg("VT's pgrp %d", tcgetpgrp(0));
-               BB_EXECVP(argv[0], argv);
-               bb_perror_msg_and_die("can't execute '%s'", argv[0]);
+               BB_EXECVP_or_die(argv);
        }
 }
 

diff --git a/coreutils/chroot.c b/coreutils/chroot.c
index bc0b1f82c4a32feb25f2d6891b9f22a6d6e76afb..046c2fabf6cbe4888a2d5a24cca67c9036d07907 100644 (file)
--- a/coreutils/chroot.c
+++ b/coreutils/chroot.c
@@ -30,6 +30,5 @@ int chroot_main(int argc UNUSED_PARAM, char **argv)
                argv[1] = (char *) "-i";
        }
 
-       BB_EXECVP(argv[0], argv);
-       bb_perror_msg_and_die("can't execute '%s'", argv[0]);
+       BB_EXECVP_or_die(argv);
 }

diff --git a/coreutils/env.c b/coreutils/env.c
index c6ba04d35482544656ce78dab4d86adb05ce9a7e..d4eab191bb54707c6873d0e58d4596c66f01d1bc 100644 (file)
--- a/coreutils/env.c
+++ b/coreutils/env.c
@@ -77,10 +77,7 @@ int env_main(int argc UNUSED_PARAM, char **argv)
        }
 
        if (argv[0]) {
-               BB_EXECVP(argv[0], argv);
-               /* SUSv3-mandated exit codes. */
-               xfunc_error_retval = (errno == ENOENT) ? 127 : 126;
-               bb_perror_msg_and_die("can't execute '%s'", argv[0]);
+               BB_EXECVP_or_die(argv);
        }
 
        if (environ) { /* clearenv() may set environ == NULL! */

diff --git a/coreutils/nice.c b/coreutils/nice.c
index 0f70f1079a4a2d1b87a6428d2a56ce15ffb4751c..ff3eb1140ee4face070fa24d111ef20b49ef453c 100644 (file)
--- a/coreutils/nice.c
+++ b/coreutils/nice.c
@@ -47,8 +47,5 @@ int nice_main(int argc, char **argv)
                }
        }
 
-       BB_EXECVP(argv[0], argv);
-       /* The exec failed... */
-       xfunc_error_retval = (errno == ENOENT) ? 127 : 126; /* SUSv3 */
-       bb_perror_msg_and_die("can't execute '%s'", argv[0]);
+       BB_EXECVP_or_die(argv);
 }

diff --git a/coreutils/nohup.c b/coreutils/nohup.c
index 1027ada1c689da32d8469990e8cd716cc9329cb4..3dc5314095594b0d763588024b8aa49773cf4e3d 100644 (file)
--- a/coreutils/nohup.c
+++ b/coreutils/nohup.c
@@ -76,6 +76,5 @@ int nohup_main(int argc UNUSED_PARAM, char **argv)
        signal(SIGHUP, SIG_IGN);
 
        argv++;
-       BB_EXECVP(argv[0], argv);
-       bb_perror_msg_and_die("can't execute '%s'", argv[0]);
+       BB_EXECVP_or_die(argv);
 }

diff --git a/include/libbb.h b/include/libbb.h
index e26001705e66f80fe02fcadf288bd9b1f67cb7d1..4b6699f2f170e612bd48c2c855bfe58f294e2cb0 100644 (file)
--- a/include/libbb.h
+++ b/include/libbb.h
@@ -839,6 +839,7 @@ int bb_execvp(const char *file, char *const argv[]) FAST_FUNC;
 #define BB_EXECVP(prog,cmd)     execvp(prog,cmd)
 #define BB_EXECLP(prog,cmd,...) execlp(prog,cmd, __VA_ARGS__)
 #endif
+int BB_EXECVP_or_die(char **argv) NORETURN FAST_FUNC;
 
 /* NOMMU friendy fork+exec: */
 pid_t spawn(char **argv) FAST_FUNC;

diff --git a/libbb/execable.c b/libbb/execable.c
index 5c7ac16a2c16465e99b1f9f3fc5f9b35e36709a0..82241cd81d5823880058d800d941e2a1190cf9c2 100644 (file)
--- a/libbb/execable.c
+++ b/libbb/execable.c
@@ -76,3 +76,11 @@ int FAST_FUNC bb_execvp(const char *file, char *const argv[])
                                        argv);
 }
 #endif
+
+int FAST_FUNC BB_EXECVP_or_die(char **argv)
+{
+       BB_EXECVP(argv[0], argv);
+       /* SUSv3-mandated exit codes */
+       xfunc_error_retval = (errno == ENOENT) ? 127 : 126;
+       bb_perror_msg_and_die("can't execute '%s'", argv[0]);
+}

diff --git a/libbb/vfork_daemon_rexec.c b/libbb/vfork_daemon_rexec.c
index 082f0f63edde4a65b13b10f1d342c2d2a109aefc..8102ea2dc3eda28cb041491257c447f6fa015122 100644 (file)
--- a/libbb/vfork_daemon_rexec.c
+++ b/libbb/vfork_daemon_rexec.c
@@ -67,40 +67,6 @@ pid_t FAST_FUNC xspawn(char **argv)
        return pid;
 }
 
-pid_t FAST_FUNC safe_waitpid(pid_t pid, int *wstat, int options)
-{
-       pid_t r;
-
-       do
-               r = waitpid(pid, wstat, options);
-       while ((r == -1) && (errno == EINTR));
-       return r;
-}
-
-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 FAST_FUNC wait4pid(pid_t pid)
-{
-       int status;
-
-       if (pid <= 0) {
-               /*errno = ECHILD; -- wrong. */
-               /* we expect errno to be already set from failed [v]fork/exec */
-               return -1;
-       }
-       if (safe_waitpid(pid, &status, 0) == -1)
-               return -1;
-       if (WIFEXITED(status))
-               return WEXITSTATUS(status);
-       if (WIFSIGNALED(status))
-               return WTERMSIG(status) + 0x180;
-       return 0;
-}
-
 #if ENABLE_FEATURE_PREFER_APPLETS
 void FAST_FUNC save_nofork_data(struct nofork_save_area *save)
 {

diff --git a/libbb/xfuncs.c b/libbb/xfuncs.c
index 65437211d084866524a1b4ed88c0c1e85e54f0a7..275dd4b627eb9cbd42bef989e8a6e87eb30bde06 100644 (file)
--- a/libbb/xfuncs.c
+++ b/libbb/xfuncs.c
@@ -268,3 +268,37 @@ int FAST_FUNC tcsetattr_stdin_TCSANOW(const struct termios *tp)
 {
        return tcsetattr(STDIN_FILENO, TCSANOW, tp);
 }
+
+pid_t FAST_FUNC safe_waitpid(pid_t pid, int *wstat, int options)
+{
+       pid_t r;
+
+       do
+               r = waitpid(pid, wstat, options);
+       while ((r == -1) && (errno == EINTR));
+       return r;
+}
+
+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 FAST_FUNC wait4pid(pid_t pid)
+{
+       int status;
+
+       if (pid <= 0) {
+               /*errno = ECHILD; -- wrong. */
+               /* we expect errno to be already set from failed [v]fork/exec */
+               return -1;
+       }
+       if (safe_waitpid(pid, &status, 0) == -1)
+               return -1;
+       if (WIFEXITED(status))
+               return WEXITSTATUS(status);
+       if (WIFSIGNALED(status))
+               return WTERMSIG(status) + 0x180;
+       return 0;
+}

diff --git a/mailutils/mail.c b/mailutils/mail.c
index 49e72c32bff54bbd9037173af9ff0189fc7c9d34..5eb99e13dae4f757340290eaf765c0d71137fa86 100644 (file)
--- a/mailutils/mail.c
+++ b/mailutils/mail.c
@@ -67,8 +67,7 @@ void FAST_FUNC launch_helper(const char **argv)
        if (!G.helper_pid) {
                // child: try to execute connection helper
                // NB: SIGCHLD & SIGALRM revert to SIG_DFL on exec
-               BB_EXECVP(argv[0], (char **)argv);
-               bb_perror_msg_and_die("can't execute '%s'", argv[0]);
+               BB_EXECVP_or_die((char**)argv);
        }
 
        // parent

diff --git a/mailutils/mime.c b/mailutils/mime.c
index 654b8731ce12abaca04fdfe003bb7d88454a75f5..5eb8ef6f2d331d41aab3a628655b04df130e7786 100644 (file)
--- a/mailutils/mime.c
+++ b/mailutils/mime.c
@@ -288,8 +288,7 @@ static int parse(const char *boundary, char **argv)
                                        xsetenv("CHARSET", charset);
                                        xsetenv("ENCODING", encoding);
                                        xsetenv("FILENAME", filename);
-                                       BB_EXECVP(argv[0], argv);
-                                       bb_perror_msg_and_die("can't execute '%s'", argv[0]);
+                                       BB_EXECVP_or_die(argv);
                                }
                                // parent dumps to fd[1]
                                close(fd[0]);

diff --git a/miscutils/chrt.c b/miscutils/chrt.c
index 3d0da58cadc95f073a35035037040138d2523a6e..d5f87c4d746738f5f113ae1fb841e09479a7ec8b 100644 (file)
--- a/miscutils/chrt.c
+++ b/miscutils/chrt.c
@@ -120,6 +120,5 @@ int chrt_main(int argc UNUSED_PARAM, char **argv)
        if (!argv[0]) /* "-p <priority> <pid> [...]" */
                goto print_rt_info;
 
-       BB_EXECVP(argv[0], argv);
-       bb_perror_msg_and_die("can't execute '%s'", argv[0]);
+       BB_EXECVP_or_die(argv);
 }

diff --git a/miscutils/ionice.c b/miscutils/ionice.c
index 8393cd8b2830fe696c21b614715dc3e7aecb7f90..52e51b9080ec3a609031de3ebaadc8c785a07bff 100644 (file)
--- a/miscutils/ionice.c
+++ b/miscutils/ionice.c
@@ -90,8 +90,7 @@ int ionice_main(int argc UNUSED_PARAM, char **argv)
                if (ioprio_set(IOPRIO_WHO_PROCESS, pid, pri) == -1)
                        bb_perror_msg_and_die("ioprio_%cet", 's');
                if (argv[0]) {
-                       BB_EXECVP(argv[0], argv);
-                       bb_perror_msg_and_die("can't execute '%s'", argv[0]);
+                       BB_EXECVP_or_die(argv);
                }
        }
 

diff --git a/miscutils/setsid.c b/miscutils/setsid.c
index 60ee062e37e4a01880acf6b99ca1e7e8eda6c8f4..c573fae343fa63bc350ebc726e72ab95b792f42b 100644 (file)
--- a/miscutils/setsid.c
+++ b/miscutils/setsid.c
@@ -45,6 +45,5 @@ int setsid_main(int argc UNUSED_PARAM, char **argv)
        }
 
        argv++;
-       BB_EXECVP(argv[0], argv);
-       bb_perror_msg_and_die("can't execute '%s'", argv[0]);
+       BB_EXECVP_or_die(argv);
 }

diff --git a/miscutils/taskset.c b/miscutils/taskset.c
index 2891003dfa69ba9b63c2ccb171e85f3c089acaee..08198d5d47a54214c45ad96940a83eaaf11255c0 100644 (file)
--- a/miscutils/taskset.c
+++ b/miscutils/taskset.c
@@ -132,6 +132,5 @@ int taskset_main(int argc UNUSED_PARAM, char **argv)
        if (!argv[0]) /* "-p <aff> <pid> [...ignored...]" */
                goto print_aff; /* print new affinity and exit */
 
-       BB_EXECVP(argv[0], argv);
-       bb_perror_msg_and_die("can't execute '%s'", argv[0]);
+       BB_EXECVP_or_die(argv);
 }

diff --git a/miscutils/time.c b/miscutils/time.c
index f5d1e15fb21f2c5fbf467e531792c4befb145223..5cfbcef8ef64202e28166f5265e49aeec64c0750 100644 (file)
--- a/miscutils/time.c
+++ b/miscutils/time.c
@@ -367,20 +367,17 @@ static void summarize(const char *fmt, char **command, resource_t *resp)
    Put the statistics in *RESP.  */
 static void run_command(char *const *cmd, resource_t *resp)
 {
-       pid_t pid;                      /* Pid of child.  */
+       pid_t pid;
        void (*interrupt_signal)(int);
        void (*quit_signal)(int);
 
        resp->elapsed_ms = monotonic_ms();
-       pid = vfork();          /* Run CMD as child process.  */
+       pid = vfork();
        if (pid < 0)
-               bb_perror_msg_and_die("fork");
-       if (pid == 0) { /* If child.  */
-               /* Don't cast execvp arguments; that causes errors on some systems,
-                  versus merely warnings if the cast is left off.  */
-               BB_EXECVP(cmd[0], cmd);
-               xfunc_error_retval = (errno == ENOENT ? 127 : 126);
-               bb_perror_msg_and_die("can't execute '%s'", cmd[0]);
+               bb_perror_msg_and_die("vfork");
+       if (pid == 0) {
+               /* Child */
+               BB_EXECVP_or_die((char**)cmd);
        }
 
        /* Have signals kill the child but not self (if possible).  */

diff --git a/miscutils/timeout.c b/miscutils/timeout.c
index 273d26953d76bd5b8508d798a936ecf71a1a6198..f6e655accc22b28635159442ba8efd4244f90f11 100644 (file)
--- a/miscutils/timeout.c
+++ b/miscutils/timeout.c
@@ -110,6 +110,5 @@ int timeout_main(int argc UNUSED_PARAM, char **argv)
        argv[0] = sv1;
        argv[1] = sv2;
 #endif
-       BB_EXECVP(argv[0], argv);
-       bb_perror_msg_and_die("can't execute '%s'", argv[0]);
+       BB_EXECVP_or_die(argv);
 }

diff --git a/networking/ifupdown.c b/networking/ifupdown.c
index 714d2a1079e6f4e188a99703ab5a61790c9d078b..1bab2c5cb77874890d42ee6725c9e415d6de4778 100644 (file)
--- a/networking/ifupdown.c
+++ b/networking/ifupdown.c
@@ -1052,8 +1052,7 @@ static int popen2(FILE **in, FILE **out, char *command, char *param)
                close(outfd.rd);
                xmove_fd(infd.rd, 0);
                xmove_fd(outfd.wr, 1);
-               BB_EXECVP(command, argv);
-               bb_perror_msg_and_die("can't execute '%s'", command);
+               BB_EXECVP_or_die(argv);
        }
        /* parent */
        close(infd.rd);

diff --git a/networking/tcpudp.c b/networking/tcpudp.c
index 4e47567389264e35fdc9482fc9fcefe00b900e46..53e622b56b186ecf46cf8d5dddc06735d01ad488 100644 (file)
--- a/networking/tcpudp.c
+++ b/networking/tcpudp.c
@@ -501,10 +501,10 @@ int tcpudpsvd_main(int argc UNUSED_PARAM, char **argv)
 #ifdef SSLSVD
        strcpy(id, utoa(pid));
        ssl_io(0, argv);
+       bb_perror_msg_and_die("can't execute '%s'", argv[0]);
 #else
-       BB_EXECVP(argv[0], argv);
+       BB_EXECVP_or_die(argv);
 #endif
-       bb_perror_msg_and_die("can't execute '%s'", argv[0]);
 }
 
 /*

diff --git a/printutils/lpd.c b/printutils/lpd.c
index 15f1ba20b20e3c0807a92c7e5fd46542efa73125..d91491f1b39d96702e73f22c0e61b602907db0a7 100644 (file)
--- a/printutils/lpd.c
+++ b/printutils/lpd.c
@@ -181,8 +181,7 @@ int lpd_main(int argc UNUSED_PARAM, char *argv[])
                        // this call reopens stdio fds to "/dev/null"
                        // (no daemonization is done)
                        bb_daemonize_or_rexec(DAEMON_DEVNULL_STDIO | DAEMON_ONLY_SANITIZE, NULL);
-                       BB_EXECVP(argv[0], argv);
-                       bb_perror_msg_and_die("can't execute '%s'", argv[0]);
+                       BB_EXECVP_or_die(argv);
                }
 
                // validate input.

diff --git a/runit/chpst.c b/runit/chpst.c
index 028a28d6c04d8c736bda092d4d405529db89808c..ad0811294619b30850e058dab028195c04cf9970 100644 (file)
--- a/runit/chpst.c
+++ b/runit/chpst.c
@@ -382,6 +382,5 @@ int chpst_main(int argc UNUSED_PARAM, char **argv)
        if (opt & OPT_2)
                close(STDERR_FILENO);
 
-       BB_EXECVP(argv[0], argv);
-       bb_perror_msg_and_die("can't execute '%s'", argv[0]);
+       BB_EXECVP_or_die(argv);
 }

diff --git a/shell/cttyhack.c b/shell/cttyhack.c
index bde2acdc93db82012a7ef4c61428d2b20454e554..67736ad620cd51ee2bbeeb61a895353f87664fc5 100644 (file)
--- a/shell/cttyhack.c
+++ b/shell/cttyhack.c
@@ -81,6 +81,5 @@ int cttyhack_main(int argc UNUSED_PARAM, char **argv)
                }
        }
 
-       BB_EXECVP(argv[0], argv);
-       bb_perror_msg_and_die("can't execute '%s'", argv[0]);
+       BB_EXECVP_or_die(argv);
 }