diff options
| -rw-r--r-- | lib/igt_core.c | 4 | ||||
| -rw-r--r-- | lib/tests/igt_fork.c | 18 |
2 files changed, 22 insertions, 0 deletions
diff --git a/lib/igt_core.c b/lib/igt_core.c index 607b34cd..425b96ff 100644 --- a/lib/igt_core.c +++ b/lib/igt_core.c @@ -1492,6 +1492,8 @@ void __igt_fail_assert(const char *domain, const char *file, const int line, */ void igt_exit(void) { + int tmp; + igt_exit_called = true; if (igt_key_file) @@ -1527,6 +1529,8 @@ void igt_exit(void) kill(test_children[c], SIGKILL); assert(!num_test_children); + assert(waitpid(-1, &tmp, WNOHANG) == -1 && errno == ECHILD); + if (!test_with_subtests) { struct timespec now; const char *result; diff --git a/lib/tests/igt_fork.c b/lib/tests/igt_fork.c index 1c10a801..fa5bb770 100644 --- a/lib/tests/igt_fork.c +++ b/lib/tests/igt_fork.c @@ -68,6 +68,20 @@ static void igt_fork_leak(void) } } +static void plain_fork_leak(void) +{ + int pid; + + switch (pid = fork()) { + case -1: + internal_assert(0); + case 0: + sleep(1); + default: + exit(0); + } +} + static void igt_fork_timeout_leak(void) { igt_fork(i, 1) { @@ -119,4 +133,8 @@ int main(int argc, char **argv) /* check that igt_waitchildren_timeout cleans up*/ ret = do_fork(igt_fork_timeout_leak); internal_assert(WEXITSTATUS(ret) == SIGKILL + 128); + + /* check that any other process leaks are caught*/ + ret = do_fork(plain_fork_leak); + internal_assert(WTERMSIG(ret) == SIGABRT); } |