summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorEric Andersen2003-05-26 18:12:00 +0000
committerEric Andersen2003-05-26 18:12:00 +0000
commitb0cfca75442293b4b670584d83c39f9cea9f5a8b (patch)
treec05e258028e66a87812e50590318b8951db4db21
parenta2d1982841adf21442c7574e784a45324e7f1db5 (diff)
downloadbusybox-b0cfca75442293b4b670584d83c39f9cea9f5a8b.zip
busybox-b0cfca75442293b4b670584d83c39f9cea9f5a8b.tar.gz
This was doing some silly stuff that is not necessary when using
vfork(), so I have simplified it.
-rw-r--r--libbb/run_parts.c40
1 files changed, 23 insertions, 17 deletions
diff --git a/libbb/run_parts.c b/libbb/run_parts.c
index 5864566..1cd13f2 100644
--- a/libbb/run_parts.c
+++ b/libbb/run_parts.c
@@ -83,31 +83,37 @@ extern int run_parts(char **args, const unsigned char test_mode)
if (test_mode & 1) {
puts(filename);
} else {
- /* exec_errno is common vfork variable */
- volatile int exec_errno = 0;
+ pid_t pid, wpid;
int result;
- int pid;
if ((pid = vfork()) < 0) {
bb_perror_msg_and_die("failed to fork");
- } else if (!pid) {
- args[0] = filename;
+ } else if (pid==0) {
execv(filename, args);
- exec_errno = errno;
_exit(1);
}
- waitpid(pid, &result, 0);
- if(exec_errno) {
- errno = exec_errno;
- bb_perror_msg_and_die("failed to exec %s", filename);
- }
- if (WIFEXITED(result) && WEXITSTATUS(result)) {
- bb_perror_msg("%s exited with return code %d", filename, WEXITSTATUS(result));
- exitstatus = 1;
- } else if (WIFSIGNALED(result)) {
- bb_perror_msg("%s exited because of uncaught signal %d", filename, WTERMSIG(result));
- exitstatus = 1;
+ /* Wait for the child process to exit. Since we use vfork
+ * we shouldn't actually have to do any waiting... */
+ wpid = wait(&result);
+ while (wpid > 0) {
+ /* Find out who died, make sure it is the right process */
+ if (pid == wpid) {
+ if (WIFEXITED(result) && WEXITSTATUS(result)) {
+ bb_perror_msg("%s exited with return code %d", filename, WEXITSTATUS(result));
+ exitstatus = 1;
+ } else if (WIFSIGNALED(result) && WIFSIGNALED(result)) {
+ int sig;
+ sig = WTERMSIG(result);
+ bb_perror_msg("%s exited because of uncaught signal %d (%s)",
+ filename, sig, u_signal_names(0, &sig, 1));
+ exitstatus = 1;
+ }
+ break;
+ } else {
+ /* Just in case some _other_ random child process exits */
+ wpid = wait(&result);
+ }
}
}
}