diff options
author | Denys Vlasenko <vda.linux@googlemail.com> | 2018-07-28 12:42:08 +0200 |
---|---|---|
committer | Denys Vlasenko <vda.linux@googlemail.com> | 2018-07-28 12:42:08 +0200 |
commit | a5db1d7354977fe295882ff557e8f5fb23741b13 (patch) | |
tree | fab0fd9d857d1e9de4ed030c03e194e18312f28f /shell/hush.c | |
parent | a8e7441176ec945a1bfb117a1067ac3a6680a19c (diff) | |
download | busybox-w32-a5db1d7354977fe295882ff557e8f5fb23741b13.tar.gz busybox-w32-a5db1d7354977fe295882ff557e8f5fb23741b13.tar.bz2 busybox-w32-a5db1d7354977fe295882ff557e8f5fb23741b13.zip |
hush: fix another case where empty "for" wasn't setting exitcode to 0
Signed-off-by: Denys Vlasenko <vda.linux@googlemail.com>
Diffstat (limited to 'shell/hush.c')
-rw-r--r-- | shell/hush.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/shell/hush.c b/shell/hush.c index 14681aa48..d17f7f29e 100644 --- a/shell/hush.c +++ b/shell/hush.c | |||
@@ -9391,11 +9391,11 @@ static int run_list(struct pipe *pi) | |||
9391 | }; /* argv list with one element: "$@" */ | 9391 | }; /* argv list with one element: "$@" */ |
9392 | char **vals; | 9392 | char **vals; |
9393 | 9393 | ||
9394 | G.last_exitcode = rcode = EXIT_SUCCESS; | ||
9394 | vals = (char**)encoded_dollar_at_argv; | 9395 | vals = (char**)encoded_dollar_at_argv; |
9395 | if (pi->next->res_word == RES_IN) { | 9396 | if (pi->next->res_word == RES_IN) { |
9396 | /* if no variable values after "in" we skip "for" */ | 9397 | /* if no variable values after "in" we skip "for" */ |
9397 | if (!pi->next->cmds[0].argv) { | 9398 | if (!pi->next->cmds[0].argv) { |
9398 | G.last_exitcode = rcode = EXIT_SUCCESS; | ||
9399 | debug_printf_exec(": null FOR: exitcode EXIT_SUCCESS\n"); | 9399 | debug_printf_exec(": null FOR: exitcode EXIT_SUCCESS\n"); |
9400 | break; | 9400 | break; |
9401 | } | 9401 | } |