diff options
| author | Denys Vlasenko <vda.linux@googlemail.com> | 2020-02-21 02:55:53 +0100 |
|---|---|---|
| committer | Denys Vlasenko <vda.linux@googlemail.com> | 2020-02-21 02:55:53 +0100 |
| commit | 3ced804e3118d138781c3e4baa6bf1589b9f2dfd (patch) | |
| tree | 963276adb3280dfa605725e20598afe566139f08 /shell/hush.c | |
| parent | cc9ecd9af13f04a4814ebdec60942962c3e9f14f (diff) | |
| download | busybox-3ced804e3118d138781c3e4baa6bf1589b9f2dfd.tar.gz | |
hush: make "exit" in trap use pre-trap exitcode - fix for nested trap
function old new delta
check_and_run_traps 276 278 +2
Signed-off-by: Denys Vlasenko <vda.linux@googlemail.com>
Diffstat (limited to 'shell/hush.c')
| -rw-r--r-- | shell/hush.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/shell/hush.c b/shell/hush.c index b881b001a..357a354e2 100644 --- a/shell/hush.c +++ b/shell/hush.c @@ -2107,16 +2107,18 @@ static int check_and_run_traps(void) if (G_traps[sig][0]) { /* We have user-defined handler */ smalluint save_rcode; + int save_pre; char *argv[3]; /* argv[0] is unused */ argv[1] = xstrdup(G_traps[sig]); /* why strdup? trap can modify itself: trap 'trap "echo oops" INT' INT */ argv[2] = NULL; + save_pre = G.pre_trap_exitcode; G.pre_trap_exitcode = save_rcode = G.last_exitcode; builtin_eval(argv); free(argv[1]); + G.pre_trap_exitcode = save_pre; G.last_exitcode = save_rcode; - G.pre_trap_exitcode = -1; # if ENABLE_HUSH_FUNCTIONS if (G.return_exitcode >= 0) { debug_printf_exec("trap exitcode:%d\n", G.return_exitcode); |