[BusyBox-cvs] busybox/libbb run_parts.c,1.6,1.7

Erik Andersen andersen at busybox.net
Mon May 26 18:12:04 UTC 2003


Update of /var/cvs/busybox/libbb
In directory winder:/tmp/cvs-serv12876/libbb

Modified Files:
	run_parts.c 
Log Message:
This was doing some silly stuff that is not necessary when using
vfork(), so I have simplified it.


Index: run_parts.c
===================================================================
RCS file: /var/cvs/busybox/libbb/run_parts.c,v
retrieving revision 1.6
retrieving revision 1.7
diff -u -d -r1.6 -r1.7
--- run_parts.c	19 Mar 2003 09:12:08 -0000	1.6
+++ run_parts.c	26 May 2003 18:12:00 -0000	1.7
@@ -83,31 +83,37 @@
 			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);
+					}
 				}
 			}
 		} 



More information about the busybox-cvs mailing list