code shrink in check_errors_in_children()

Signed-off-by: Denys Vlasenko <vda.linux@googlemail.com>

1 files changed, 8 insertions, 3 deletions

diff --git a/archival/libarchive/open_transformer.c b/archival/libarchive/open_transformer.c
index dae04aa57..4e44a87e9 100644
--- a/archival/libarchive/open_transformer.c
+++ b/archival/libarchive/open_transformer.c
@@ -34,6 +34,7 @@ void check_errors_in_children(int signo)
         if (!signo) {
                 /* block waiting for any child */
                 if (wait(&status) < 0)
+//FIXME: check EINTR?
                         return; /* probably there are no children */
                 goto check_status;
         }
@@ -41,14 +42,18 @@ void check_errors_in_children(int signo)
         /* Wait for any child without blocking */
         for (;;) {
                 if (wait_any_nohang(&status) < 0)
+//FIXME: check EINTR?
                         /* wait failed?! I'm confused... */
                         return;
  check_status:
-                if (WIFEXITED(status) && WEXITSTATUS(status) == 0)
+                /*if (WIFEXITED(status) && WEXITSTATUS(status) == 0)*/
+                /* On Linux, the above can be checked simply as: */
+                if (status == 0)
                         /* this child exited with 0 */
                         continue;
-                /* Cannot happen?
-                if (!WIFSIGNALED(status) && !WIFEXITED(status)) ???; */
+                /* Cannot happen:
+                if (!WIFSIGNALED(status) && !WIFEXITED(status)) ???;
+                 */
                 bb_got_signal = 1;
         }
 }