diff options
author | Lennart Poettering <lennart@poettering.net> | 2013-12-23 19:10:11 +0100 |
---|---|---|
committer | Lennart Poettering <lennart@poettering.net> | 2013-12-23 19:10:11 +0100 |
commit | fbadf04511389c4a0687ba5e9baf0ecebdbb07f1 (patch) | |
tree | e54a8fe180ebe121a40c70cf23024c3fee77fd3e /src/machine | |
parent | e7f7a1b0222c49fd4706a77bbdac59ac28a4ddbb (diff) |
bus: when getting a kdbus connection into a container wait first for child, then read message
There's no EOF generated for AF_UNIX/SOCK_DGRAM sockets, hence let's
wait for the child first to see if it succeeded, only then read the socket.
Diffstat (limited to 'src/machine')
-rw-r--r-- | src/machine/machinectl.c | 18 |
1 files changed, 9 insertions, 9 deletions
diff --git a/src/machine/machinectl.c b/src/machine/machinectl.c index a2af87c17d..ab9060ab99 100644 --- a/src/machine/machinectl.c +++ b/src/machine/machinectl.c @@ -451,6 +451,15 @@ static int openpt_in_namespace(pid_t pid, int flags) { close_nointr_nofail(pair[1]); pair[1] = -1; + r = wait_for_terminate(child, &si); + if (r < 0 || si.si_code != CLD_EXITED || si.si_status != EXIT_SUCCESS || master < 0) { + + if (master >= 0) + close_nointr_nofail(master); + + return r < 0 ? r : -EIO; + } + if (recvmsg(pair[0], &mh, MSG_NOSIGNAL|MSG_CMSG_CLOEXEC) < 0) return -errno; @@ -470,15 +479,6 @@ static int openpt_in_namespace(pid_t pid, int flags) { master = fds[0]; } - r = wait_for_terminate(child, &si); - if (r < 0 || si.si_code != CLD_EXITED || si.si_status != EXIT_SUCCESS || master < 0) { - - if (master >= 0) - close_nointr_nofail(master); - - return r < 0 ? r : -EIO; - } - return master; } |