diff options
-rw-r--r-- | ChangeLog | 4 | ||||
-rw-r--r-- | posix/tst-spawn.c | 20 |
2 files changed, 16 insertions, 8 deletions
@@ -1,3 +1,7 @@ +2017-10-12 Szabolcs Nagy <szabolcs.nagy@arm.com> + + * posix/tst-spawn.c (do_test): Wait for both children. + 2017-10-12 H.J. Lu <hongjiu.lu@intel.com> [BZ #22284] diff --git a/posix/tst-spawn.c b/posix/tst-spawn.c index 08d92bd7a7..4e5e76351c 100644 --- a/posix/tst-spawn.c +++ b/posix/tst-spawn.c @@ -23,9 +23,10 @@ #include <spawn.h> #include <stdlib.h> #include <string.h> -#include <unistd.h> #include <wait.h> #include <sys/param.h> +#include <support/check.h> +#include <support/xunistd.h> /* Nonzero if the program gets called via `exec'. */ @@ -249,13 +250,16 @@ do_test (int argc, char *argv[]) error (EXIT_FAILURE, errno, "posix_spawn_file_actions_destroy"); free (name3_copy); - /* Wait for the child. */ - if (waitpid (pid, &status, 0) != pid) - error (EXIT_FAILURE, errno, "wrong child"); + /* Wait for the children. */ + TEST_VERIFY (xwaitpid (pid, &status, 0) == pid); + TEST_VERIFY (WIFEXITED (status)); + TEST_VERIFY (!WIFSIGNALED (status)); + TEST_VERIFY (WEXITSTATUS (status) == 0); - if (WTERMSIG (status) != 0) - error (EXIT_FAILURE, 0, "Child terminated incorrectly"); - status = WEXITSTATUS (status); + xwaitpid (-1, &status, 0); + TEST_VERIFY (WIFEXITED (status)); + TEST_VERIFY (!WIFSIGNALED (status)); + TEST_VERIFY (WEXITSTATUS (status) == 0); - return status; + return 0; } |