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 /applets/applet_tables.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 'applets/applet_tables.c')
0 files changed, 0 insertions, 0 deletions
