Preparing for release of 3.0.0pre1

[rsync/rsync.git] / clientserver.c

diff --git a/clientserver.c b/clientserver.c
index 43bfdf4..700a126 100644 (file)
--- a/clientserver.c
+++ b/clientserver.c
@@ -23,7 +23,6 @@
 #include "ifuncs.h"
 
 extern int verbose;
-extern int quiet;
 extern int output_motd;
 extern int list_only;
 extern int am_sender;
@@ -840,9 +839,70 @@ int start_daemon(int f_in, int f_out)
        return rsync_module(f_in, f_out, i, addr, host);
 }
 
+static void create_pid_file(void)
+{
+       char *pid_file = lp_pid_file();
+       char pidbuf[16];
+       pid_t pid = getpid();
+       int fd;
+
+       if (!pid_file || !*pid_file)
+               return;
+
+       cleanup_set_pid(pid);
+       if ((fd = do_open(pid_file, O_WRONLY|O_CREAT|O_EXCL, 0666 & ~orig_umask)) == -1) {
+               cleanup_set_pid(0);
+               fprintf(stderr, "failed to create pid file %s: %s\n", pid_file, strerror(errno));
+               rsyserr(FLOG, errno, "failed to create pid file %s", pid_file);
+               exit_cleanup(RERR_FILEIO);
+       }
+       snprintf(pidbuf, sizeof pidbuf, "%ld\n", (long)pid);
+       write(fd, pidbuf, strlen(pidbuf));
+       close(fd);
+}
+
+/* Become a daemon, discarding the controlling terminal. */
+static void become_daemon(void)
+{
+       int i;
+       pid_t pid = fork();
+
+       if (pid) {
+               if (pid < 0) {
+                       fprintf(stderr, "failed to fork: %s\n", strerror(errno));
+                       exit_cleanup(RERR_FILEIO);
+               }
+               _exit(0);
+       }
+
+       create_pid_file();
+
+       /* detach from the terminal */
+#ifdef HAVE_SETSID
+       setsid();
+#elif defined TIOCNOTTY
+       i = open("/dev/tty", O_RDWR);
+       if (i >= 0) {
+               ioctl(i, (int)TIOCNOTTY, (char *)0);
+               close(i);
+       }
+#endif
+       /* make sure that stdin, stdout an stderr don't stuff things
+        * up (library functions, for example) */
+       for (i = 0; i < 3; i++) {
+               close(i);
+               open("/dev/null", O_RDWR);
+       }
+}
+
 int daemon_main(void)
 {
-       char *pid_file;
+       if (!config_file) {
+               if (am_server && am_root <= 0)
+                       config_file = RSYNCD_USERCONF;
+               else
+                       config_file = RSYNCD_SYSCONF;
+       }
 
        if (is_a_socket(STDIN_FILENO)) {
                int i;
@@ -858,11 +918,15 @@ int daemon_main(void)
                return start_daemon(STDIN_FILENO, STDIN_FILENO);
        }
 
-       if (!no_detach)
-               become_daemon();
-
-       if (!lp_load(config_file, 1))
+       if (!lp_load(config_file, 1)) {
+               fprintf(stderr, "Failed to parse config file: %s\n", config_file);
                exit_cleanup(RERR_SYNTAX);
+       }
+
+       if (no_detach)
+               create_pid_file();
+       else
+               become_daemon();
 
        if (rsync_port == 0 && (rsync_port = lp_rsync_port()) == 0)
                rsync_port = RSYNC_PORT;
@@ -877,23 +941,6 @@ int daemon_main(void)
         * address too.  In fact, why not just do inet_ntop on the
         * local address??? */
 
-       if (((pid_file = lp_pid_file()) != NULL) && (*pid_file != '\0')) {
-               char pidbuf[16];
-               int fd;
-               pid_t pid = getpid();
-               cleanup_set_pid(pid);
-               if ((fd = do_open(lp_pid_file(), O_WRONLY|O_CREAT|O_TRUNC,
-                                       0666 & ~orig_umask)) == -1) {
-                       cleanup_set_pid(0);
-                       rsyserr(FLOG, errno, "failed to create pid file %s",
-                               pid_file);
-                       exit_cleanup(RERR_FILEIO);
-               }
-               snprintf(pidbuf, sizeof pidbuf, "%ld\n", (long)pid);
-               write(fd, pidbuf, strlen(pidbuf));
-               close(fd);
-       }
-
        start_accept_loop(rsync_port, start_daemon);
        return -1;
 }