diff options
author | Bernhard Reutner-Fischer <rep.dot.nop@gmail.com> | 2006-05-30 12:16:54 +0000 |
---|---|---|
committer | Bernhard Reutner-Fischer <rep.dot.nop@gmail.com> | 2006-05-30 12:16:54 +0000 |
commit | c58dbf28e06b68a852028663b342eca98e739ab5 (patch) | |
tree | 5723f3087eb5acc5ee759cd125d97c7d64f7ddca | |
parent | 3ab3080bc2c52216b5f4fb2ffbef74ec5e6c1d86 (diff) | |
download | busybox-w32-c58dbf28e06b68a852028663b342eca98e739ab5.tar.gz busybox-w32-c58dbf28e06b68a852028663b342eca98e739ab5.tar.bz2 busybox-w32-c58dbf28e06b68a852028663b342eca98e739ab5.zip |
- remove useless variable "status" from init_main.
http://www.opengroup.org/onlinepubs/009695399/functions/waitpid.html suggests
that we need not specify a status if we don't want, and we don't.
"If wait() or waitpid() return because the status of a child process is available, these functions shall return a value equal to the process ID of the child process. In this case, if the value of the argument stat_loc is not a null pointer, information shall be stored in the location pointed to by stat_loc. "
text data bss dec hex filename
5391 32 8 5431 1537 init/init.o.06
5379 32 8 5419 152b init/init.o
-rw-r--r-- | init/init.c | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/init/init.c b/init/init.c index 9cbe8189a..b17ebc2ce 100644 --- a/init/init.c +++ b/init/init.c | |||
@@ -1005,7 +1005,6 @@ int init_main(int argc, char **argv) | |||
1005 | { | 1005 | { |
1006 | struct init_action *a; | 1006 | struct init_action *a; |
1007 | pid_t wpid; | 1007 | pid_t wpid; |
1008 | int status; | ||
1009 | 1008 | ||
1010 | if (argc > 1 && !strcmp(argv[1], "-q")) { | 1009 | if (argc > 1 && !strcmp(argv[1], "-q")) { |
1011 | return kill(1,SIGHUP); | 1010 | return kill(1,SIGHUP); |
@@ -1139,7 +1138,7 @@ int init_main(int argc, char **argv) | |||
1139 | sleep(1); | 1138 | sleep(1); |
1140 | 1139 | ||
1141 | /* Wait for a child process to exit */ | 1140 | /* Wait for a child process to exit */ |
1142 | wpid = wait(&status); | 1141 | wpid = wait(NULL); |
1143 | while (wpid > 0) { | 1142 | while (wpid > 0) { |
1144 | /* Find out who died and clean up their corpse */ | 1143 | /* Find out who died and clean up their corpse */ |
1145 | for (a = init_action_list; a; a = a->next) { | 1144 | for (a = init_action_list; a; a = a->next) { |
@@ -1153,7 +1152,7 @@ int init_main(int argc, char **argv) | |||
1153 | } | 1152 | } |
1154 | } | 1153 | } |
1155 | /* see if anyone else is waiting to be reaped */ | 1154 | /* see if anyone else is waiting to be reaped */ |
1156 | wpid = waitpid (-1, &status, WNOHANG); | 1155 | wpid = waitpid (-1, NULL, WNOHANG); |
1157 | } | 1156 | } |
1158 | } | 1157 | } |
1159 | } | 1158 | } |