ash: return exit status of nofork applets (again)
authorRon Yorston <rmy@pobox.com>
Thu, 20 Feb 2020 08:23:03 +0000 (08:23 +0000)
committerDenys Vlasenko <vda.linux@googlemail.com>
Thu, 20 Feb 2020 09:39:46 +0000 (10:39 +0100)
commitd5bfe26c457a97a8b525b6c27f5498525a5829bf
treea80b55274ac3fa64dd93244064e75a875d27abdd
parentecc85832f8b4d8278b0a17aa62b0c8a9f81461e4
ash: return exit status of nofork applets (again)

Since commit d81af7216 (ash: eval: Reap zombies after built-in commands
and functions) if the shell is compiled with SH_STANDALONE and SH_NOFORK
enabled nofork applets potentially return the incorrect status.

The status value returned by evalcommand() in this case is obtained from
exitstatus in a call to waitforjob(NULL).  This overwrites the status
set for nonfork applets.

If this commit seems familiar it's essentially a reversion of commit
5ccb0e92fa (ash: return exit status of nofork applets).  What was correct
in 2016 is wrong in 2020 and vice versa.

Signed-off-by: Ron Yorston <rmy@pobox.com>
Signed-off-by: Denys Vlasenko <vda.linux@googlemail.com>
shell/ash.c