emo/busybox
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Avoid calling exit() from within fork/vfork'ed processes.

Browse Source 
-Erik
This commit is contained in:
Eric Andersen 2002-09-26 13:59:40 +00:00
parent a920871c3a
commit 6f2ebcaa01
1 changed files with 7 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
shell/lash.c
View File

@ -1214,7 +1214,7 @@ static int pseudo_exec(struct child_prog *child)
*/ */
for (x = bltins; x->cmd; x++) { for (x = bltins; x->cmd; x++) {
if (strcmp(child->argv[0], x->cmd) == 0 ) { if (strcmp(child->argv[0], x->cmd) == 0 ) {
exit(x->function(child)); _exit(x->function(child));
} }
} }
@ -1222,7 +1222,7 @@ static int pseudo_exec(struct child_prog *child)
for (x = bltins_forking; x->cmd; x++) { for (x = bltins_forking; x->cmd; x++) {
if (strcmp(child->argv[0], x->cmd) == 0) { if (strcmp(child->argv[0], x->cmd) == 0) {
applet_name=x->cmd; applet_name=x->cmd;
exit (x->function(child)); _exit (x->function(child));
} }
} }
#ifdef CONFIG_FEATURE_SH_STANDALONE_SHELL #ifdef CONFIG_FEATURE_SH_STANDALONE_SHELL
@ -1258,7 +1258,11 @@ static int pseudo_exec(struct child_prog *child)
#endif #endif
execvp(child->argv[0], child->argv); execvp(child->argv[0], child->argv);
perror_msg_and_die("%s", child->argv[0]);
/* Do not use perror_msg_and_die() here, since we must not
* call exit() but should call _exit() instead */
fprintf(stderr, "%s: %s\n", child->argv[0], strerror(err));
_exit(EXIT_FAILURE);
} }
static void insert_job(struct job *newjob, int inbg) static void insert_job(struct job *newjob, int inbg)