r22417: Refactor the various daemon run-mode options to make the semantics
authorJames Peach <jpeach@samba.org>
Fri, 20 Apr 2007 18:34:33 +0000 (18:34 +0000)
committerGerald (Jerry) Carter <jerry@samba.org>
Wed, 10 Oct 2007 17:19:33 +0000 (12:19 -0500)
of the various flags explicit.
(This used to be commit 19c929c6330a50f278ac322ac5fcb83d03734ea2)

source3/include/popt_common.h
source3/libsmb/namequery.c
source3/nmbd/nmbd.c
source3/nmbd/nmbd_lmhosts.c
source3/nsswitch/winbindd.c
source3/smbd/server.c

index 4c3facb48f9cff9addb0683c78d0b54c79e53bce..9db5ecc3d13d2a24da7fabf7040f8745083b598c 100644 (file)
@@ -50,6 +50,17 @@ struct user_auth_info {
        int signing_state;
 };
 
+enum smb_server_mode {
+       /* Daemonize and manage our own sockets */
+       SERVER_MODE_DAEMON,
+       /* Don't daemonize or manage sockets */
+       SERVER_MODE_INETD,
+       /* Don't daemonize, but do manage sockets */
+       SERVER_MODE_FOREGROUND,
+       /* Run in the foreground, log to stdout, don't fork children */
+       SERVER_MODE_INTERACTIVE
+};
+
 extern struct user_auth_info cmdline_auth_info;
 
 #endif /* _POPT_COMMON_H */
index cbd94ff5672885e749a3ecd90b9abaaec12ab4ca..0826bc52181debb327244406bb71bd5b1497b001 100644 (file)
@@ -651,7 +651,7 @@ struct in_addr *name_query(int fd,const char *name,int name_type,
  Start parsing the lmhosts file.
 *********************************************************/
 
-XFILE *startlmhosts(char *fname)
+XFILE *startlmhosts(const char *fname)
 {
        XFILE *fp = x_fopen(fname,O_RDONLY, 0);
        if (!fp) {
index 46f209872b03555d4bad081affa96b9e85f12dad..3a18e667740e95b131ebc22c94830016c34dcb38 100644 (file)
@@ -33,15 +33,6 @@ extern BOOL global_in_nmbd;
 
 extern BOOL override_logfile;
 
-/* are we running as a daemon ? */
-static BOOL is_daemon;
-
-/* fork or run in foreground ? */
-static BOOL Fork = True;
-
-/* log to standard output ? */
-static BOOL log_stdout;
-
 /* have we found LanMan clients yet? */
 BOOL found_lm_clients = False;
 
@@ -578,7 +569,7 @@ static void process(void)
  Open the socket communication.
  **************************************************************************** */
 
-static BOOL open_sockets(BOOL isdaemon, int port)
+static BOOL open_sockets(enum smb_server_mode server_mode, int port)
 {
        /*
         * The sockets opened here will be used to receive broadcast
@@ -588,12 +579,13 @@ static BOOL open_sockets(BOOL isdaemon, int port)
         * now deprecated.
         */
 
-       if ( isdaemon )
+       if ( server_mode == SERVER_MODE_INETD ) {
+               ClientNMB = 0;
+       } else {
                ClientNMB = open_socket_in(SOCK_DGRAM, port,
                                           0, interpret_addr(lp_socket_address()),
                                           True);
-       else
-               ClientNMB = 0;
+       }
   
        ClientDGRAM = open_socket_in(SOCK_DGRAM, DGRAM_PORT,
                                           3, interpret_addr(lp_socket_address()),
@@ -622,15 +614,20 @@ static BOOL open_sockets(BOOL isdaemon, int port)
  int main(int argc, const char *argv[])
 {
        pstring logfile;
-       static BOOL opt_interactive;
        poptContext pc;
-       static char *p_lmhosts = dyn_LMHOSTSFILE;
-       static BOOL no_process_group = False;
+       const char *p_lmhosts = dyn_LMHOSTSFILE;
+       BOOL no_process_group = False;
+       BOOL log_stdout = False;
+       enum smb_server_mode server_mode = SERVER_MODE_DAEMON;
+
        struct poptOption long_options[] = {
        POPT_AUTOHELP
-       {"daemon", 'D', POPT_ARG_VAL, &is_daemon, True, "Become a daemon(default)" },
-       {"interactive", 'i', POPT_ARG_VAL, &opt_interactive, True, "Run interactive (not a daemon)" },
-       {"foreground", 'F', POPT_ARG_VAL, &Fork, False, "Run daemon in foreground (for daemontools & etc)" },
+       {"daemon", 'D', POPT_ARG_VAL, &server_mode, SERVER_MODE_DAEMON,
+               "Become a daemon(default)" },
+       {"interactive", 'i', POPT_ARG_VAL, &server_mode,
+               SERVER_MODE_INTERACTIVE, "Run interactive (not a daemon)" },
+       {"foreground", 'F', POPT_ARG_VAL, &server_mode,
+               SERVER_MODE_FOREGROUND, "Run daemon in foreground (for daemontools & etc)" },
        {"no-process-group", 0, POPT_ARG_VAL, &no_process_group, True, "Don't create a new process group" },
        {"log-stdout", 'S', POPT_ARG_VAL, &log_stdout, True, "Log to stdout" },
        {"hosts", 'H', POPT_ARG_STRING, &p_lmhosts, 'H', "Load a netbios hosts file"},
@@ -680,12 +677,11 @@ static BOOL open_sockets(BOOL isdaemon, int port)
        BlockSignals(True, SIGUSR2);
 #endif
 
-       if ( opt_interactive ) {
-               Fork = False;
+       if (server_mode == SERVER_MODE_INTERACTIVE) {
                log_stdout = True;
        }
 
-       if ( log_stdout && Fork ) {
+       if (log_stdout && server_mode == SERVER_MODE_DAEMON) {
                DEBUG(0,("ERROR: Can't log to stdout (-S) unless daemon is in foreground (-F) or interactive (-i)\n"));
                exit(1);
        }
@@ -712,14 +708,19 @@ static BOOL open_sockets(BOOL isdaemon, int port)
 
        set_samba_nb_type();
 
-       if (!is_daemon && !is_a_socket(0)) {
-               DEBUG(0,("standard input is not a socket, assuming -D option\n"));
-               is_daemon = True;
+       if (is_a_socket(0)) {
+               if (server_mode == SERVER_MODE_DAEMON) {
+                       DEBUG(0,("standard input is a socket, "
+                                   "assuming -F option\n"));
+               }
+               server_mode = SERVER_MODE_INETD;
        }
-  
-       if (is_daemon && !opt_interactive) {
+
+       if (server_mode == SERVER_MODE_DAEMON) {
                DEBUG( 2, ( "Becoming a daemon.\n" ) );
-               become_daemon(Fork, no_process_group);
+               become_daemon(True, no_process_group);
+       } else if (server_mode == SERVER_MODE_FOREGROUND) {
+               become_daemon(False, no_process_group);
        }
 
 #if HAVE_SETPGID
@@ -727,7 +728,7 @@ static BOOL open_sockets(BOOL isdaemon, int port)
         * If we're interactive we want to set our own process group for 
         * signal management.
         */
-       if (opt_interactive && !no_process_group)
+       if (server_mode == SERVER_MODE_INTERACTIVE && !no_process_group)
                setpgid( (pid_t)0, (pid_t)0 );
 #endif
 
@@ -758,7 +759,7 @@ static BOOL open_sockets(BOOL isdaemon, int port)
 
        DEBUG( 3, ( "Opening sockets %d\n", global_nmb_port ) );
 
-       if ( !open_sockets( is_daemon, global_nmb_port ) ) {
+       if ( !open_sockets( server_mode, global_nmb_port ) ) {
                kill_async_dns_child();
                return 1;
        }
index b14e13f3a47e15da9142148ad7b8d4a0eacd1b35..be3ddfe637aafe0dd5d0dd172db8546e91df8341 100644 (file)
@@ -29,7 +29,7 @@
 Load a lmhosts file.
 ****************************************************************************/
 
-void load_lmhosts_file(char *fname)
+void load_lmhosts_file(const char *fname)
 {  
        pstring name;
        int name_type;
index fcedaebb273e9314c083103cf95c9485699af8fa..da0473583f5bd3183a0a236046ff7b274d836ea6 100644 (file)
@@ -30,7 +30,6 @@
 #define DBGC_CLASS DBGC_WINBIND
 
 BOOL opt_nocache = False;
-static BOOL interactive = False;
 
 extern BOOL override_logfile;
 
@@ -911,15 +910,17 @@ static void process_loop(void)
 int main(int argc, char **argv, char **envp)
 {
        pstring logfile;
-       static BOOL Fork = True;
-       static BOOL log_stdout = False;
-       static BOOL no_process_group = False;
+       BOOL log_stdout = False;
+       BOOL no_process_group = False;
+
+       enum smb_server_mode server_mode = SERVER_MODE_DAEMON;
+
        struct poptOption long_options[] = {
                POPT_AUTOHELP
                { "stdout", 'S', POPT_ARG_VAL, &log_stdout, True, "Log to stdout" },
-               { "foreground", 'F', POPT_ARG_VAL, &Fork, False, "Daemon in foreground mode" },
+               { "foreground", 'F', POPT_ARG_VAL, &server_mode, SERVER_MODE_FOREGROUND, "Daemon in foreground mode" },
                { "no-process-group", 0, POPT_ARG_VAL, &no_process_group, True, "Don't create a new process group" },
-               { "interactive", 'i', POPT_ARG_NONE, NULL, 'i', "Interactive mode" },
+               { "interactive", 'i', POPT_ARG_VAL, &server_mode, SERVER_MODE_INTERACTIVE, "Interactive mode" },
                { "no-caching", 'n', POPT_ARG_VAL, &opt_nocache, True, "Disable caching" },
                POPT_COMMON_SAMBA
                POPT_TABLEEND
@@ -957,20 +958,17 @@ int main(int argc, char **argv, char **envp)
        pc = poptGetContext("winbindd", argc, (const char **)argv, long_options,
                                                POPT_CONTEXT_KEEP_FIRST);
 
-       while ((opt = poptGetNextOpt(pc)) != -1) {
-               switch (opt) {
-                       /* Don't become a daemon */
-               case 'i':
-                       interactive = True;
-                       log_stdout = True;
-                       Fork = False;
-                       break;
+       while ((opt = poptGetNextOpt(pc)) != -1) {}
+
+       if (server_mode == SERVER_MODE_INTERACTIVE) {
+               log_stdout = True;
+               if (DEBUGLEVEL >= 9) {
+                       talloc_enable_leak_report();
                }
        }
 
-
-       if (log_stdout && Fork) {
-               printf("Can't log to stdout (-S) unless daemon is in foreground +(-F) or interactive (-i)\n");
+       if (log_stdout && server_mode == SERVER_MODE_DAEMON) {
+               printf("Can't log to stdout (-S) unless daemon is in foreground (-F) or interactive (-i)\n");
                poptPrintUsage(pc, stderr, 0);
                exit(1);
        }
@@ -1041,8 +1039,12 @@ int main(int argc, char **argv, char **envp)
        CatchSignal(SIGUSR2, sigusr2_handler);         /* Debugging sigs */
        CatchSignal(SIGHUP, sighup_handler);
 
-       if (!interactive)
-               become_daemon(Fork, no_process_group);
+       if (server_mode == SERVER_MODE_DAEMON) {
+               DEBUG( 3, ( "Becoming a daemon.\n" ) );
+               become_daemon(True, no_process_group);
+       } else if (server_mode == SERVER_MODE_FOREGROUND) {
+               become_daemon(False, no_process_group);
+       }
 
        pidfile_create("winbindd");
 
@@ -1067,8 +1069,9 @@ int main(int argc, char **argv, char **envp)
         * If we're interactive we want to set our own process group for
         * signal management.
         */
-       if (interactive && !no_process_group)
+       if (server_mode == SERVER_MODE_INTERACTIVE && !no_process_group) {
                setpgid( (pid_t)0, (pid_t)0);
+       }
 #endif
 
        TimeInit();
index f1efcd41e91209a374d9a2c3ecd4c89db0796100..255f9d0c6574adaa305b664b9b9de553c723fd36 100644 (file)
@@ -300,7 +300,7 @@ static BOOL allowable_number_of_smbd_processes(void)
  Open the socket communication.
 ****************************************************************************/
 
-static BOOL open_sockets_smbd(BOOL is_daemon, BOOL interactive, const char *smb_ports)
+static BOOL open_sockets_smbd(enum smb_server_mode server_mode, const char *smb_ports)
 {
        int num_interfaces = iface_count();
        int num_sockets = 0;
@@ -311,11 +311,10 @@ static BOOL open_sockets_smbd(BOOL is_daemon, BOOL interactive, const char *smb_
        int i;
        char *ports;
 
-       if (!is_daemon) {
+       if (server_mode == SERVER_MODE_INETD) {
                return open_sockets_inetd();
        }
 
-               
 #ifdef HAVE_ATEXIT
        {
                static int atexit_set;
@@ -531,8 +530,13 @@ static BOOL open_sockets_smbd(BOOL is_daemon, BOOL interactive, const char *smb_
                        /* Ensure child is set to blocking mode */
                        set_blocking(smbd_server_fd(),True);
 
-                       if (smbd_server_fd() != -1 && interactive)
+                       /* In interactive mode, return with a connected socket.
+                        * Foreground and daemon modes should fork worker
+                        * processes.
+                        */
+                       if (server_mode == SERVER_MODE_INTERACTIVE) {
                                return True;
+                       }
                        
                        if (allowable_number_of_smbd_processes() &&
                            smbd_server_fd() != -1 &&
@@ -857,22 +861,25 @@ extern void build_options(BOOL screen);
  int main(int argc,const char *argv[])
 {
        /* shall I run as a daemon */
-       static BOOL is_daemon = False;
-       static BOOL interactive = False;
-       static BOOL Fork = True;
-       static BOOL no_process_group = False;
-       static BOOL log_stdout = False;
-       static char *ports = NULL;
-       static char *profile_level = NULL;
+       BOOL no_process_group = False;
+       BOOL log_stdout = False;
+       const char *ports = NULL;
+       const char *profile_level = NULL;
        int opt;
        poptContext pc;
 
+       enum smb_server_mode server_mode = SERVER_MODE_DAEMON;
+
        struct poptOption long_options[] = {
        POPT_AUTOHELP
-       {"daemon", 'D', POPT_ARG_VAL, &is_daemon, True, "Become a daemon (default)" },
-       {"interactive", 'i', POPT_ARG_VAL, &interactive, True, "Run interactive (not a daemon)"},
-       {"foreground", 'F', POPT_ARG_VAL, &Fork, False, "Run daemon in foreground (for daemontools, etc.)" },
-       {"no-process-group", '\0', POPT_ARG_VAL, &no_process_group, True, "Don't create a new process group" },
+       {"daemon", 'D', POPT_ARG_VAL, &server_mode, SERVER_MODE_DAEMON,
+               "Become a daemon (default)" },
+       {"interactive", 'i', POPT_ARG_VAL, &server_mode, SERVER_MODE_INTERACTIVE,
+               "Run interactive (not a daemon)"},
+       {"foreground", 'F', POPT_ARG_VAL, &server_mode, SERVER_MODE_FOREGROUND,
+               "Run daemon in foreground (for daemontools, etc.)" },
+       {"no-process-group", '\0', POPT_ARG_VAL, &no_process_group, True,
+               "Don't create a new process group" },
        {"log-stdout", 'S', POPT_ARG_VAL, &log_stdout, True, "Log to stdout" },
        {"build-options", 'b', POPT_ARG_NONE, NULL, 'b', "Print build options" },
        {"port", 'p', POPT_ARG_STRING, &ports, 0, "Listen on the specified ports"},
@@ -912,16 +919,14 @@ extern void build_options(BOOL screen);
 
        set_remote_machine_name("smbd", False);
 
-       if (interactive) {
-               Fork = False;
+       if (server_mode == SERVER_MODE_INTERACTIVE) {
                log_stdout = True;
+               if (DEBUGLEVEL >= 9) {
+                       talloc_enable_leak_report();
+               }
        }
 
-       if (interactive && (DEBUGLEVEL >= 9)) {
-               talloc_enable_leak_report();
-       }
-
-       if (log_stdout && Fork) {
+       if (log_stdout && server_mode == SERVER_MODE_DAEMON) {
                DEBUG(0,("ERROR: Can't log to stdout (-S) unless daemon is in foreground (-F) or interactive (-i)\n"));
                exit(1);
        }
@@ -1011,21 +1016,19 @@ extern void build_options(BOOL screen);
 
        DEBUG(3,( "loaded services\n"));
 
-       if (!is_daemon && !is_a_socket(0)) {
-               if (!interactive)
-                       DEBUG(0,("standard input is not a socket, assuming -D option\n"));
-
-               /*
-                * Setting is_daemon here prevents us from eventually calling
-                * the open_sockets_inetd()
-                */
-
-               is_daemon = True;
+       if (is_a_socket(0)) {
+               if (server_mode == SERVER_MODE_DAEMON) {
+                       DEBUG(0,("standard input is a socket, "
+                                   "assuming -F option\n"));
+               }
+               server_mode = SERVER_MODE_INETD;
        }
 
-       if (is_daemon && !interactive) {
+       if (server_mode == SERVER_MODE_DAEMON) {
                DEBUG( 3, ( "Becoming a daemon.\n" ) );
-               become_daemon(Fork, no_process_group);
+               become_daemon(True, no_process_group);
+       } else if (server_mode == SERVER_MODE_FOREGROUND) {
+               become_daemon(False, no_process_group);
        }
 
 #if HAVE_SETPGID
@@ -1033,15 +1036,18 @@ extern void build_options(BOOL screen);
         * If we're interactive we want to set our own process group for
         * signal management.
         */
-       if (interactive && !no_process_group)
+       if (server_mode == SERVER_MODE_INTERACTIVE && !no_process_group) {
                setpgid( (pid_t)0, (pid_t)0);
+       }
 #endif
 
        if (!directory_exist(lp_lockdir(), NULL))
                mkdir(lp_lockdir(), 0755);
 
-       if (is_daemon)
+       if (server_mode != SERVER_MODE_INETD &&
+           server_mode != SERVER_MODE_INTERACTIVE) {
                pidfile_create("smbd");
+       }
 
        /* Setup all the TDB's - including CLEAR_IF_FIRST tdb's. */
        if (!message_init())
@@ -1099,9 +1105,10 @@ extern void build_options(BOOL screen);
           running as a daemon -- bad things will happen if
           smbd is launched via inetd and we fork a copy of 
           ourselves here */
-
-       if ( is_daemon && !interactive )
+       if (server_mode != SERVER_MODE_INETD &&
+           server_mode != SERVER_MODE_INTERACTIVE) {
                start_background_queue(); 
+       }
 
        /* Always attempt to initialize DMAPI. We will only use it later if
         * lp_dmapi_support is set on the share, but we need a single global
@@ -1109,8 +1116,9 @@ extern void build_options(BOOL screen);
         */
        dmapi_init_session();
 
-       if (!open_sockets_smbd(is_daemon, interactive, ports))
+       if (!open_sockets_smbd(server_mode, ports)) {
                exit(1);
+       }
 
        /*
         * everything after this point is run after the fork()
@@ -1123,7 +1131,8 @@ extern void build_options(BOOL screen);
        /* Possibly reload the services file. Only worth doing in
         * daemon mode. In inetd mode, we know we only just loaded this.
         */
-       if (is_daemon) {
+       if (server_mode != SERVER_MODE_INETD &&
+           server_mode != SERVER_MODE_INTERACTIVE) {
                reload_services(True);
        }