s3-testparm: Throw warning when 'workgroup' and 'netbios name' are identical.
[kai/samba.git] / source3 / utils / testparm.c
index a74c2eaa973c84e14f48a2c1bd0d5d8cbca6fef3..73e173e20975f7803aeb6fdac49423a045d82840 100644 (file)
@@ -8,7 +8,7 @@
    
    This program is free software; you can redistribute it and/or modify
    it under the terms of the GNU General Public License as published by
-   the Free Software Foundation; either version 2 of the License, or
+   the Free Software Foundation; either version 3 of the License, or
    (at your option) any later version.
    
    This program is distributed in the hope that it will be useful,
@@ -17,8 +17,7 @@
    GNU General Public License for more details.
    
    You should have received a copy of the GNU General Public License
-   along with this program; if not, write to the Free Software
-   Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.
+   along with this program.  If not, see <http://www.gnu.org/licenses/>.
 */
 
 /*
 
 #include "includes.h"
 
-extern BOOL AllowDebugChange;
+extern bool AllowDebugChange;
+
+/*******************************************************************
+ Check if a directory exists.
+********************************************************************/
+
+static bool directory_exist_stat(char *dname,SMB_STRUCT_STAT *st)
+{
+       SMB_STRUCT_STAT st2;
+       bool ret;
+
+       if (!st)
+               st = &st2;
+
+       if (sys_stat(dname, st, false) != 0)
+               return(False);
+
+       ret = S_ISDIR(st->st_ex_mode);
+       if(!ret)
+               errno = ENOTDIR;
+       return ret;
+}
 
 /***********************************************
  Here we do a set of 'hard coded' checks for bad
@@ -57,48 +77,71 @@ cannot be set in the smb.conf file. nmbd will abort with this setting.\n");
                ret = 1;
        }
 
-       if (!directory_exist(lp_lockdir(), &st)) {
+       if (strequal(lp_workgroup(), global_myname())) {
+               fprintf(stderr, "WARNING: 'workgroup' and 'netbios name' " \
+                       "must differ.\n");
+               ret = 1;
+       }
+
+       if (!directory_exist_stat(lp_lockdir(), &st)) {
                fprintf(stderr, "ERROR: lock directory %s does not exist\n",
                       lp_lockdir());
                ret = 1;
-       } else if ((st.st_mode & 0777) != 0755) {
+       } else if ((st.st_ex_mode & 0777) != 0755) {
                fprintf(stderr, "WARNING: lock directory %s should have permissions 0755 for browsing to work\n",
                       lp_lockdir());
                ret = 1;
        }
 
-       if (!directory_exist(lp_piddir(), &st)) {
+       if (!directory_exist_stat(lp_statedir(), &st)) {
+               fprintf(stderr, "ERROR: state directory %s does not exist\n",
+                      lp_statedir());
+               ret = 1;
+       } else if ((st.st_ex_mode & 0777) != 0755) {
+               fprintf(stderr, "WARNING: state directory %s should have permissions 0755 for browsing to work\n",
+                      lp_statedir());
+               ret = 1;
+       }
+
+       if (!directory_exist_stat(lp_cachedir(), &st)) {
+               fprintf(stderr, "ERROR: cache directory %s does not exist\n",
+                      lp_cachedir());
+               ret = 1;
+       } else if ((st.st_ex_mode & 0777) != 0755) {
+               fprintf(stderr, "WARNING: cache directory %s should have permissions 0755 for browsing to work\n",
+                      lp_cachedir());
+               ret = 1;
+       }
+
+       if (!directory_exist_stat(lp_piddir(), &st)) {
                fprintf(stderr, "ERROR: pid directory %s does not exist\n",
                       lp_piddir());
                ret = 1;
        }
 
+       if (lp_passdb_expand_explicit()) {
+               fprintf(stderr, "WARNING: passdb expand explicit = yes is "
+                       "deprecated\n");
+       }
+
        /*
         * Password server sanity checks.
         */
 
        if((lp_security() == SEC_SERVER || lp_security() >= SEC_DOMAIN) && !lp_passwordserver()) {
-               pstring sec_setting;
+               const char *sec_setting;
                if(lp_security() == SEC_SERVER)
-                       pstrcpy(sec_setting, "server");
+                       sec_setting = "server";
                else if(lp_security() == SEC_DOMAIN)
-                       pstrcpy(sec_setting, "domain");
+                       sec_setting = "domain";
+               else
+                       sec_setting = "";
 
                fprintf(stderr, "ERROR: The setting 'security=%s' requires the 'password server' parameter be set \
 to a valid password server.\n", sec_setting );
                ret = 1;
        }
 
-       
-       /*
-        * Check 'hosts equiv' and 'use rhosts' compatibility with 'hostname lookup' value.
-        */
-
-       if(*lp_hosts_equiv() && !lp_hostname_lookups()) {
-               fprintf(stderr, "ERROR: The setting 'hosts equiv = %s' requires that 'hostname lookups = yes'.\n", lp_hosts_equiv());
-               ret = 1;
-       }
-
        /*
         * Password chat sanity checks.
         */
@@ -113,21 +156,23 @@ to a valid password server.\n", sec_setting );
                if (!lp_pam_password_change()) {
 #endif
 
-                       if(lp_passwd_program() == NULL) {
+                       if((lp_passwd_program() == NULL) ||
+                          (strlen(lp_passwd_program()) == 0))
+                       {
                                fprintf( stderr, "ERROR: the 'unix password sync' parameter is set and there is no valid 'passwd program' \
 parameter.\n" );
                                ret = 1;
                        } else {
-                               pstring passwd_prog;
-                               pstring truncated_prog;
+                               const char *passwd_prog;
+                               char *truncated_prog = NULL;
                                const char *p;
 
-                               pstrcpy( passwd_prog, lp_passwd_program());
+                               passwd_prog = lp_passwd_program();
                                p = passwd_prog;
-                               *truncated_prog = '\0';
-                               next_token(&p, truncated_prog, NULL, sizeof(pstring));
-
-                               if(access(truncated_prog, F_OK) == -1) {
+                               next_token_talloc(talloc_tos(),
+                                               &p,
+                                               &truncated_prog, NULL);
+                               if (truncated_prog && access(truncated_prog, F_OK) == -1) {
                                        fprintf(stderr, "ERROR: the 'unix password sync' parameter is set and the 'passwd program' (%s) \
 cannot be executed (error was %s).\n", truncated_prog, strerror(errno) );
                                        ret = 1;
@@ -144,13 +189,23 @@ parameter.\n");
                        ret = 1;
                }
 
+               if ((lp_passwd_program() != NULL) &&
+                   (strlen(lp_passwd_program()) > 0))
+               {
+                       /* check if there's a %u parameter present */
+                       if(strstr_m(lp_passwd_program(), "%u") == NULL) {
+                               fprintf(stderr, "ERROR: the 'passwd program' (%s) requires a '%%u' parameter.\n", lp_passwd_program());
+                               ret = 1;
+                       }
+               }
+
                /*
                 * Check that we have a valid script and that it hasn't
                 * been written to expect the old password.
                 */
 
                if(lp_encrypted_passwords()) {
-                       if(strstr( lp_passwd_chat(), "%o")!=NULL) {
+                       if(strstr_m( lp_passwd_chat(), "%o")!=NULL) {
                                fprintf(stderr, "ERROR: the 'passwd chat' script [%s] expects to use the old plaintext password \
 via the %%o substitution. With encrypted passwords this is not possible.\n", lp_passwd_chat() );
                                ret = 1;
@@ -186,39 +241,131 @@ via the %%o substitution. With encrypted passwords this is not possible.\n", lp_
        if (!lp_passdb_backend()) {
                fprintf(stderr,"ERROR: passdb backend must have a value or be left out\n");
        }
+       
+       if (lp_os_level() > 255) {
+               fprintf(stderr,"WARNING: Maximum value for 'os level' is 255!\n");      
+       }
 
        return ret;
 }   
 
+/**
+ * per-share logic tests
+ */
+static void do_per_share_checks(int s)
+{
+       const char **deny_list = lp_hostsdeny(s);
+       const char **allow_list = lp_hostsallow(s);
+       int i;
+
+       if(deny_list) {
+               for (i=0; deny_list[i]; i++) {
+                       char *hasstar = strchr_m(deny_list[i], '*');
+                       char *hasquery = strchr_m(deny_list[i], '?');
+                       if(hasstar || hasquery) {
+                               fprintf(stderr,"Invalid character %c in hosts deny list (%s) for service %s.\n",
+                                          hasstar ? *hasstar : *hasquery, deny_list[i], lp_servicename(s) );
+                       }
+               }
+       }
+
+       if(allow_list) {
+               for (i=0; allow_list[i]; i++) {
+                       char *hasstar = strchr_m(allow_list[i], '*');
+                       char *hasquery = strchr_m(allow_list[i], '?');
+                       if(hasstar || hasquery) {
+                               fprintf(stderr,"Invalid character %c in hosts allow list (%s) for service %s.\n",
+                                          hasstar ? *hasstar : *hasquery, allow_list[i], lp_servicename(s) );
+                       }
+               }
+       }
+
+       if(lp_level2_oplocks(s) && !lp_oplocks(s)) {
+               fprintf(stderr,"Invalid combination of parameters for service %s. \
+                          Level II oplocks can only be set if oplocks are also set.\n",
+                          lp_servicename(s) );
+       }
+
+       if (lp_map_hidden(s) && !(lp_create_mask(s) & S_IXOTH)) {
+               fprintf(stderr,"Invalid combination of parameters for service %s. \
+                          Map hidden can only work if create mask includes octal 01 (S_IXOTH).\n",
+                          lp_servicename(s) );
+       }
+       if (lp_map_hidden(s) && (lp_force_create_mode(s) & S_IXOTH)) {
+               fprintf(stderr,"Invalid combination of parameters for service %s. \
+                          Map hidden can only work if force create mode excludes octal 01 (S_IXOTH).\n",
+                          lp_servicename(s) );
+       }
+       if (lp_map_system(s) && !(lp_create_mask(s) & S_IXGRP)) {
+               fprintf(stderr,"Invalid combination of parameters for service %s. \
+                          Map system can only work if create mask includes octal 010 (S_IXGRP).\n",
+                          lp_servicename(s) );
+       }
+       if (lp_map_system(s) && (lp_force_create_mode(s) & S_IXGRP)) {
+               fprintf(stderr,"Invalid combination of parameters for service %s. \
+                          Map system can only work if force create mode excludes octal 010 (S_IXGRP).\n",
+                          lp_servicename(s) );
+       }
+#ifdef HAVE_CUPS
+       if (lp_printing(s) == PRINT_CUPS && *(lp_printcommand(s)) != '\0') {
+                fprintf(stderr,"Warning: Service %s defines a print command, but \
+rameter is ignored when using CUPS libraries.\n",
+                          lp_servicename(s) );
+       }
+#endif
+}
+
  int main(int argc, const char *argv[])
 {
-       const char *config_file = dyn_CONFIGFILE;
+       const char *config_file = get_dyn_CONFIGFILE();
        int s;
-       static BOOL silent_mode = False;
+       static int silent_mode = False;
+       static int show_all_parameters = False;
        int ret = 0;
        poptContext pc;
-       static const char *term_code = "";
+       static char *parameter_name = NULL;
+       static const char *section_name = NULL;
        static char *new_local_machine = NULL;
        const char *cname;
        const char *caddr;
        static int show_defaults;
+       static int skip_logic_checks = 0;
 
        struct poptOption long_options[] = {
                POPT_AUTOHELP
                {"suppress-prompt", 's', POPT_ARG_VAL, &silent_mode, 1, "Suppress prompt for enter"},
                {"verbose", 'v', POPT_ARG_NONE, &show_defaults, 1, "Show default options too"},
                {"server", 'L',POPT_ARG_STRING, &new_local_machine, 0, "Set %%L macro to servername\n"},
-               {"encoding", 't', POPT_ARG_STRING, &term_code, 0, "Print parameters with encoding"},
+               {"skip-logic-checks", 'l', POPT_ARG_NONE, &skip_logic_checks, 1, "Skip the global checks"},
+               {"show-all-parameters", '\0', POPT_ARG_VAL, &show_all_parameters, True, "Show the parameters, type, possible values" },
+               {"parameter-name", '\0', POPT_ARG_STRING, &parameter_name, 0, "Limit testparm to a named parameter" },
+               {"section-name", '\0', POPT_ARG_STRING, &section_name, 0, "Limit testparm to a named section" },
                POPT_COMMON_VERSION
+               POPT_COMMON_DEBUGLEVEL
                POPT_TABLEEND
        };
 
+       TALLOC_CTX *frame = talloc_stackframe();
+
+       load_case_tables();
+       /*
+        * Set the default debug level to 2.
+        * Allow it to be overridden by the command line,
+        * not by smb.conf.
+        */
+       DEBUGLEVEL_CLASS[DBGC_ALL] = 2;
+
        pc = poptGetContext(NULL, argc, argv, long_options, 
                            POPT_CONTEXT_KEEP_FIRST);
        poptSetOtherOptionHelp(pc, "[OPTION...] <config-file> [host-name] [host-ip]");
 
        while(poptGetNextOpt(pc) != -1);
 
+       if (show_all_parameters) {
+               show_parameter_list();
+               exit(0);
+       }
+
        setup_logging(poptGetArg(pc), True);
 
        if (poptPeekArg(pc)) 
@@ -227,9 +374,12 @@ via the %%o substitution. With encrypted passwords this is not possible.\n", lp_
        cname = poptGetArg(pc);
        caddr = poptGetArg(pc);
 
+       poptFreeContext(pc);
+
        if ( cname && ! caddr ) {
                printf ( "ERROR: You must specify both a machine name and an IP address.\n" );
-               return(1);
+               ret = 1;
+               goto done;
        }
 
        if (new_local_machine) {
@@ -237,19 +387,22 @@ via the %%o substitution. With encrypted passwords this is not possible.\n", lp_
        }
 
        dbf = x_stderr;
-       DEBUGLEVEL = 2;
+       /* Don't let the debuglevel be changed by smb.conf. */
        AllowDebugChange = False;
 
        fprintf(stderr,"Load smb config files from %s\n",config_file);
 
-       if (!lp_load(config_file,False,True,False)) {
+       if (!lp_load_with_registry_shares(config_file,False,True,False,True)) {
                fprintf(stderr,"Error loading services.\n");
-               return(1);
+               ret = 1;
+               goto done;
        }
 
        fprintf(stderr,"Loaded services file OK.\n");
 
-       ret = do_global_checks();
+       if (skip_logic_checks == 0) {
+               ret = do_global_checks();
+       }
 
        for (s=0;s<1000;s++) {
                if (VALID_SNUM(s))
@@ -262,81 +415,14 @@ via the %%o substitution. With encrypted passwords this is not possible.\n", lp_
        }
 
        for (s=0;s<1000;s++) {
-               if (VALID_SNUM(s)) {
-                       const char **deny_list = lp_hostsdeny(s);
-                       const char **allow_list = lp_hostsallow(s);
-                       int i;
-                       if(deny_list) {
-                               for (i=0; deny_list[i]; i++) {
-                                       char *hasstar = strchr_m(deny_list[i], '*');
-                                       char *hasquery = strchr_m(deny_list[i], '?');
-                                       if(hasstar || hasquery) {
-                                               fprintf(stderr,"Invalid character %c in hosts deny list (%s) for service %s.\n",
-                                                          hasstar ? *hasstar : *hasquery, deny_list[i], lp_servicename(s) );
-                                       }
-                               }
-                       }
-
-                       if(allow_list) {
-                               for (i=0; allow_list[i]; i++) {
-                                       char *hasstar = strchr_m(allow_list[i], '*');
-                                       char *hasquery = strchr_m(allow_list[i], '?');
-                                       if(hasstar || hasquery) {
-                                               fprintf(stderr,"Invalid character %c in hosts allow list (%s) for service %s.\n",
-                                                          hasstar ? *hasstar : *hasquery, allow_list[i], lp_servicename(s) );
-                                       }
-                               }
-                       }
-
-                       if(lp_level2_oplocks(s) && !lp_oplocks(s)) {
-                               fprintf(stderr,"Invalid combination of parameters for service %s. \
-                                          Level II oplocks can only be set if oplocks are also set.\n",
-                                          lp_servicename(s) );
-                       }
-
-                       if (lp_map_hidden(s) && !(lp_create_mask(s) & S_IXOTH)) {
-                               fprintf(stderr,"Invalid combination of parameters for service %s. \
-                                          Map hidden can only work if create mask includes octal 01 (S_IXOTH).\n",
-                                          lp_servicename(s) );
-                       }
-                       if (lp_map_hidden(s) && (lp_force_create_mode(s) & S_IXOTH)) {
-                               fprintf(stderr,"Invalid combination of parameters for service %s. \
-                                          Map hidden can only work if force create mode excludes octal 01 (S_IXOTH).\n",
-                                          lp_servicename(s) );
-                       }
-                       if (lp_map_system(s) && !(lp_create_mask(s) & S_IXGRP)) {
-                               fprintf(stderr,"Invalid combination of parameters for service %s. \
-                                          Map system can only work if create mask includes octal 010 (S_IXGRP).\n",
-                                          lp_servicename(s) );
-                       }
-                       if (lp_map_system(s) && (lp_force_create_mode(s) & S_IXGRP)) {
-                               fprintf(stderr,"Invalid combination of parameters for service %s. \
-                                          Map system can only work if force create mode excludes octal 010 (S_IXGRP).\n",
-                                          lp_servicename(s) );
-                       }
+               if (VALID_SNUM(s) && (skip_logic_checks == 0)) {
+                       do_per_share_checks(s);
                }
        }
 
 
-       if (!silent_mode) {
-               fprintf(stderr,"Server role: ");
-               switch(lp_server_role()) {
-                       case ROLE_STANDALONE:
-                               fprintf(stderr,"ROLE_STANDALONE\n");
-                               break;
-                       case ROLE_DOMAIN_MEMBER:
-                               fprintf(stderr,"ROLE_DOMAIN_MEMBER\n");
-                               break;
-                       case ROLE_DOMAIN_BDC:
-                               fprintf(stderr,"ROLE_DOMAIN_BDC\n");
-                               break;
-                       case ROLE_DOMAIN_PDC:
-                               fprintf(stderr,"ROLE_DOMAIN_PDC\n");
-                               break;
-                       default:
-                               fprintf(stderr,"Unknown -- internal error?\n");
-                               break;
-               }
+       if (!section_name && !parameter_name) {
+               fprintf(stderr,"Server role: %s\n", server_role_str(lp_server_role()));
        }
 
        if (!cname) {
@@ -345,6 +431,36 @@ via the %%o substitution. With encrypted passwords this is not possible.\n", lp_
                        fflush(stdout);
                        getc(stdin);
                }
+               if (parameter_name || section_name) {
+                       bool isGlobal = False;
+                       s = GLOBAL_SECTION_SNUM;
+
+                       if (!section_name) {
+                               section_name = GLOBAL_NAME;
+                               isGlobal = True;
+                       } else if ((isGlobal=!strwicmp(section_name, GLOBAL_NAME)) == 0 &&
+                                (s=lp_servicenumber(section_name)) == -1) {
+                                       fprintf(stderr,"Unknown section %s\n",
+                                               section_name);
+                                       ret = 1;
+                                       goto done;
+                       }
+                       if (parameter_name) {
+                               if (!dump_a_parameter( s, parameter_name, stdout, isGlobal)) {
+                                       fprintf(stderr,"Parameter %s unknown for section %s\n",
+                                               parameter_name, section_name);
+                                       ret = 1;
+                                       goto done;
+                               }
+                       } else {
+                               if (isGlobal == True)
+                                       lp_dump(stdout, show_defaults, 0);
+                               else
+                                       lp_dump_one(stdout, show_defaults, s);
+                       }
+                       goto done;
+               }
+
                lp_dump(stdout, show_defaults, lp_numservices());
        }
 
@@ -363,5 +479,10 @@ via the %%o substitution. With encrypted passwords this is not possible.\n", lp_
                        }
                }
        }
-       return(ret);
+
+done:
+       gfree_loadparm();
+       TALLOC_FREE(frame);
+       return ret;
 }
+