s3-popt: Only include popt-common.h when needed.
[nivanova/samba-autobuild/.git] / source3 / utils / testparm.c
index dd4cef8e23b1e2beafe4302025575c643905e308..6d606bba41cda2794663f956a9ce754b474c82da 100644 (file)
  */
 
 #include "includes.h"
+#include "popt_common.h"
 
 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
  configuration settings.
@@ -56,17 +78,43 @@ 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;
@@ -202,6 +250,72 @@ via the %%o substitution. With encrypted passwords this is not possible.\n", lp_
        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 = get_dyn_CONFIGFILE();
@@ -210,10 +324,8 @@ via the %%o substitution. With encrypted passwords this is not possible.\n", lp_
        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;
@@ -223,19 +335,24 @@ via the %%o substitution. With encrypted passwords this is not possible.\n", lp_
                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);
@@ -256,24 +373,24 @@ 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);
-       }
-
-       if (new_local_machine) {
-               set_local_machine_name(new_local_machine, True);
+               ret = 1;
+               goto done;
        }
 
        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_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");
@@ -294,64 +411,7 @@ via the %%o substitution. With encrypted passwords this is not possible.\n", lp_
 
        for (s=0;s<1000;s++) {
                if (VALID_SNUM(s) && (skip_logic_checks == 0)) {
-                       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 \
-print command parameter is ignored when using CUPS libraries.\n",
-                                          lp_servicename(s) );
-                       }
-#endif
+                       do_per_share_checks(s);
                }
        }
 
@@ -377,13 +437,15 @@ print command parameter is ignored when using CUPS libraries.\n",
                                 (s=lp_servicenumber(section_name)) == -1) {
                                        fprintf(stderr,"Unknown section %s\n",
                                                section_name);
-                                       return(1);
+                                       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);
-                                       return(1);
+                                       ret = 1;
+                                       goto done;
                                }
                        } else {
                                if (isGlobal == True)
@@ -391,7 +453,7 @@ print command parameter is ignored when using CUPS libraries.\n",
                                else
                                        lp_dump_one(stdout, show_defaults, s);
                        }
-                       return(ret);
+                       goto done;
                }
 
                lp_dump(stdout, show_defaults, lp_numservices());
@@ -412,7 +474,10 @@ print command parameter is ignored when using CUPS libraries.\n",
                        }
                }
        }
+
+done:
+       gfree_loadparm();
        TALLOC_FREE(frame);
-       return(ret);
+       return ret;
 }