d/freebsd-nq
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

PR 28852

Browse Source 
sh -e behaviour was incorrect when && and || statements where used in
"if" clauses.

This is the patch submitted by MORI Kouji <mori@tri.asanuma.co.jp>.

It fixes the issue at hand, but sh fixes like this are super-hard to
verify that they don't break anything else.  I ran some of my old test
cases and a few big GNU configure scripts that detected mistakes
before, with the previous sh, patched sh and bash.  No differences in
behaviour found.  MFC recommended after longer than usual time.

Compiles on i386 and sledge.
This commit is contained in:
Martin Cracauer 2003-12-28 02:05:12 +00:00
parent 5caf2b00f0
commit bd80d26f3b
1 changed files with 1 additions and 24 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

25
bin/sh/eval.c
View File

@ -205,7 +205,6 @@ evaltree(union node *n, int flags)
case NAND:
evaltree(n->nbinary.ch1, EV_TESTED);
if (evalskip || exitstatus != 0) {
flags |= EV_TESTED;
goto out;
}
evaltree(n->nbinary.ch2, flags);
@ -246,25 +245,9 @@ evaltree(union node *n, int flags)
break;
case NFOR:
evalfor(n);
/*
* The 'for' command does not set exitstatus, so the value
* now in exitstatus is from the last command executed in
* the 'for' loop. That exit value had been tested (wrt
* 'sh -e' checking) while processing that command, and
* it should not be re-tested here.
*/
flags |= EV_TESTED;
break;
case NCASE:
evalcase(n, flags);
/*
* The 'case' command does not set exitstatus, so the value
* now in exitstatus is from the last command executed in
* the 'case' block. That exit value had been tested (wrt
* 'sh -e' checking) while processing that command, and
* it should not be re-tested here.
*/
flags |= EV_TESTED;
break;
case NDEFUN:
defun(n->narg.text, n->narg.next);
@ -289,14 +272,8 @@ evaltree(union node *n, int flags)
out:
if (pendingsigs)
dotrap();
/*
* XXX - Like "!(n->type == NSEMI)", more types will probably
* need to be excluded from this test. It's probably better
* to set or unset EV_TESTED in the loop above than to bloat
* the conditional here.
*/
if ((flags & EV_EXIT) || (eflag && exitstatus
&& !(flags & EV_TESTED) && !(n->type == NSEMI)))
&& !(flags & EV_TESTED) && (n->type == NCMD)))
exitshell(exitstatus);
}