From 898f4da326051addf1ec17887e6b1737d0dab2ec Mon Sep 17 00:00:00 2001 From: Mike Yuan Date: Tue, 11 Jul 2023 01:46:29 +0800 Subject: shared/async: don't use WEXITED for waitpid() Follow-up for c26d7837bb08508c8d906d849dff8f1bc465063e waitpid() doesn't support WEXITED and returns -1 (EINVAL), which results in the intermediate close process not getting reaped. Fixes https://github.com/systemd/systemd/issues/26744#issuecomment-1628240782 --- src/shared/async.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) (limited to 'src/shared/async.c') diff --git a/src/shared/async.c b/src/shared/async.c index 1cf1936d4b..6af63d7a9c 100644 --- a/src/shared/async.c +++ b/src/shared/async.c @@ -103,10 +103,9 @@ int asynchronous_close(int fd) { * * We usually prefer calling waitid(), but before kernel 4.7 it didn't support __WCLONE while * waitpid() did. Hence let's use waitpid() here, it's good enough for our purposes here. */ - for (;;) { - if (waitpid(pid, NULL, WEXITED|__WCLONE) >= 0 || errno != EINTR) + for (;;) + if (waitpid(pid, NULL, __WCLONE) >= 0 || errno != EINTR) break; - } } return -EBADF; /* return an invalidated fd */ -- cgit v1.2.3