[PATCH] clear_traps: reset savestatus

[Martijn Dekker]

Op 27-11-18 om 17:24 schreef Martijn Dekker:
Big bad bug: it appears that subshells always return status 0 in traps.

As posted elsewhere, looks like the problem is simply that savestatus 
("/* exit status of last command outside traps */") isn't reset to -1 
upon resetting traps when forking a subshell.

Reposting patch for Patchwork purposes:

diff --git a/src/trap.c b/src/trap.c
index ab0ecd4..7740955 100644
--- a/src/trap.c
+++ b/src/trap.c
@@ -168,6 +168,7 @@ clear_traps(void)
                }
        }
        trapcnt = 0;
+       savestatus = -1;
        INTON;
 }


Thanks,

- M.