diff options
author | Lennart Poettering <lennart@poettering.net> | 2007-09-12 00:04:21 +0000 |
---|---|---|
committer | Lennart Poettering <lennart@poettering.net> | 2007-09-12 00:04:21 +0000 |
commit | 7f92542420ef6085b6f090954052266cc70af8a1 (patch) | |
tree | 86640b0497bf7f34c0c7df5b6c36bdf6ec78adea /src/daemon/main.c | |
parent | 54506ab44a9ac61e79b3a5c632db56bde41beded (diff) |
consolidate close() calls to pa_close(), and make sure on every occasion that we handle failures of close() sensibly
git-svn-id: file:///home/lennart/svn/public/pulseaudio/branches/lennart@1811 fefdeb5f-60dc-0310-8127-8f9354f1896f
Diffstat (limited to 'src/daemon/main.c')
-rw-r--r-- | src/daemon/main.c | 16 |
1 files changed, 8 insertions, 8 deletions
diff --git a/src/daemon/main.c b/src/daemon/main.c index 4509e7f9..0bbddffa 100644 --- a/src/daemon/main.c +++ b/src/daemon/main.c @@ -160,9 +160,9 @@ static void signal_callback(pa_mainloop_api*m, PA_GCC_UNUSED pa_signal_event *e, static void close_pipe(int p[2]) { if (p[0] != -1) - close(p[0]); + pa_assert_se(pa_close(p[0]) == 0); if (p[1] != -1) - close(p[1]); + pa_assert_se(pa_close(p[1]) == 0); p[0] = p[1] = -1; } @@ -537,7 +537,7 @@ int main(int argc, char *argv[]) { if (child != 0) { /* Father */ - close(daemon_pipe[1]); + pa_assert_se(pa_close(daemon_pipe[1]) == 0); daemon_pipe[1] = -1; if (pa_loop_read(daemon_pipe[0], &retval, sizeof(retval), NULL) != sizeof(retval)) { @@ -553,7 +553,7 @@ int main(int argc, char *argv[]) { goto finish; } - close(daemon_pipe[0]); + pa_assert_se(pa_close(daemon_pipe[0]) == 0); daemon_pipe[0] = -1; #endif @@ -568,9 +568,9 @@ int main(int argc, char *argv[]) { #endif #ifndef OS_IS_WIN32 - close(0); - close(1); - close(2); + pa_close(0); + pa_close(1); + pa_close(2); open("/dev/null", O_RDONLY); open("/dev/null", O_WRONLY); @@ -592,7 +592,7 @@ int main(int argc, char *argv[]) { #ifdef TIOCNOTTY if ((tty_fd = open("/dev/tty", O_RDWR)) >= 0) { ioctl(tty_fd, TIOCNOTTY, (char*) 0); - close(tty_fd); + pa_assert_se(pa_close(tty_fd) == 0); } #endif } |