After logging (if enabled) is switched to file, there is no reason to keep

stdin/stdout/stderr active, so dup them to /dev/null.
This commit is contained in:
Nicholas Marriott 2009-06-26 22:12:19 +00:00
parent b11b1dd1c0
commit 46f80d01ff

View File

@ -25,6 +25,7 @@
#include <errno.h>
#include <fcntl.h>
#include <paths.h>
#include <signal.h>
#include <stdio.h>
#include <stdlib.h>
@ -131,7 +132,7 @@ server_client_index(struct client *c)
int
server_start(char *path)
{
int pair[2], srv_fd;
int pair[2], srv_fd, null_fd;
char *cause;
char rpathbuf[MAXPATHLEN];
@ -176,6 +177,18 @@ server_start(char *path)
}
logfile("server");
/*
* Close stdin/stdout/stderr. Can't let daemon() do this as they are
* needed until now to print configuration file errors.
*/
if ((null_fd = open(_PATH_DEVNULL, O_RDWR)) != -1) {
dup2(null_fd, STDIN_FILENO);
dup2(null_fd, STDOUT_FILENO);
dup2(null_fd, STDERR_FILENO);
if (null_fd > 2)
close(null_fd);
}
log_debug("server started, pid %ld", (long) getpid());
log_debug("socket path %s", socket_path);