diff options
author | Gerrit Pape <pape@smarden.org> | 2010-06-28 17:11:58 +1000 |
---|---|---|
committer | Herbert Xu <herbert@gondor.apana.org.au> | 2010-06-28 17:11:58 +1000 |
commit | cdfb42fd67558847b689c69b6fdd23016e7220eb (patch) | |
tree | 220fce3cad122e3b3851472be68c76424315f380 /src | |
parent | [BUILTIN] Continue after EINTR in read(1) with no pending signals (diff) | |
download | dash-cdfb42fd67558847b689c69b6fdd23016e7220eb.tar.gz dash-cdfb42fd67558847b689c69b6fdd23016e7220eb.zip |
[EVAL] Don't clear eflag in evalbackcmd
According to http://www.opengroup.org/onlinepubs/009695399/utilities/xcu_chap02.html#tag_02_12 "A subshell environment shall be created as a duplicate of the shell environment, except that signal traps set by that shell environment shall be set to the default values." Currently the eflag is cleared when forking a subshell, e.g. $ dash -c 'set -e ; z=$(false;echo foo) ; echo $z' foo With this commit the eflag is preserved for subshells, and dash exits 1 before echo. The problem was reported by Vincent Lefevre through http://bugs.debian.org/514863 Signed-off-by: Gerrit Pape <pape@smarden.org> Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
Diffstat (limited to '')
-rw-r--r-- | src/eval.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/src/eval.c b/src/eval.c index 439f881..d142412 100644 --- a/src/eval.c +++ b/src/eval.c @@ -625,7 +625,6 @@ evalbackcmd(union node *n, struct backcmd *result) dup2(pip[1], 1); close(pip[1]); } - eflag = 0; evaltreenr(n, EV_EXIT); /* NOTREACHED */ } |