diff options
author | Colin Guthrie <cguthrie@mandriva.org> | 2011-03-25 09:12:51 +0000 |
---|---|---|
committer | Colin Guthrie <cguthrie@mandriva.org> | 2011-03-25 21:35:40 +0000 |
commit | 34ddc5b9c533ed83d53238fff0777663f9c50336 (patch) | |
tree | 0f274a06d544cc68daa9154b06e26a9a8d5b5370 /src | |
parent | f49711c99a86075de0a2337c4ea71049a9aa146a (diff) |
daemon: Fix some more error paths in the double forking.
As spotted by Tanu Kaskinen:
The first process: daemon_pipe is not closed if the first fork() call
fails. Even if it doesn't fail, the first process never closes
daemon_pipe[0].
The second process: daemon_pipe[1] is not closed if anything fails
between the first and the second fork() call. Also, if the second fork
fails, then the finish section writes to daemon_pipe2[1], even though
only the third process should do that. Also, if anything fails between
the first and the second fork, then the second process never writes
anything to daemon_pipe[1]. I don't know what happens in the first
process in this case - does it get an error or does pa_loop_read() get
stuck.
The third process: No problems :)
Diffstat (limited to 'src')
-rw-r--r-- | src/daemon/main.c | 9 |
1 files changed, 8 insertions, 1 deletions
diff --git a/src/daemon/main.c b/src/daemon/main.c index b77a8b61..f939313c 100644 --- a/src/daemon/main.c +++ b/src/daemon/main.c @@ -729,6 +729,7 @@ int main(int argc, char *argv[]) { if ((child = fork()) < 0) { pa_log(_("fork() failed: %s"), pa_cstrerror(errno)); + pa_close_pipe(daemon_pipe); goto finish; } @@ -793,6 +794,7 @@ int main(int argc, char *argv[]) { if ((child = fork()) < 0) { pa_log(_("fork() failed: %s"), pa_cstrerror(errno)); + pa_close_pipe(daemon_pipe2); goto finish; } @@ -1128,10 +1130,15 @@ finish: pa_signal_done(); #ifdef HAVE_FORK - if (daemon_pipe2[1] >= 0) + /* If we have daemon_pipe[1] still open, this means we've failed after + * the first fork, but before the second. Therefore just write to it. */ + if (daemon_pipe[1] >= 0) + pa_loop_write(daemon_pipe[1], &retval, sizeof(retval), NULL); + else if (daemon_pipe2[1] >= 0) pa_loop_write(daemon_pipe2[1], &retval, sizeof(retval), NULL); pa_close_pipe(daemon_pipe2); + pa_close_pipe(daemon_pipe); #endif if (mainloop) |