diff options
author | Denys Vlasenko <vda.linux@googlemail.com> | 2018-05-24 15:26:28 +0200 |
---|---|---|
committer | Denys Vlasenko <vda.linux@googlemail.com> | 2018-05-24 15:29:15 +0200 |
commit | ccb8e4bc4fb74701d0d323d61e9359d8597a4272 (patch) | |
tree | ebab2c7ac66d99102978efa66045d33abfb16341 /libbb/obscure.c | |
parent | 8c24af9dcfe0d89a660c39838eec4d20bc13efbf (diff) | |
download | busybox-ccb8e4bc4fb74701d0d323d61e9359d8597a4272.tar.gz |
fsck: fix incorrect handling of child exit
In commit
c4fb8c6a - fsck: do not use statics
not only statics were changed but also a couple of
statics-unrelated changes were made.
This included the handling of the child termination status
as follows:
- if (WIFEXITED(status))
- status = WEXITSTATUS(status);
- else if (WIFSIGNALED(status)) {
+ status = WEXITSTATUS(status);
+ if (WIFSIGNALED(status)) {
This is wrong, should have used a different variable to hold exit code.
Reported by Niklas Hambüchen <mail@nh2.me>.
function old new delta
wait_one 294 282 -12
Signed-off-by: Denys Vlasenko <vda.linux@googlemail.com>
Diffstat (limited to 'libbb/obscure.c')
0 files changed, 0 insertions, 0 deletions