s3:utils: Remove pointless if-clause for remote_machine
[metze/samba/wip.git] / source3 / utils / testparm.c
index 7e49c0806426784a600b5f84c08444e8f1bc4ad9..a53eea5b8cc6668c4f6ddbf4a30360b26e82449d 100644 (file)
@@ -1,21 +1,21 @@
-/* 
+/*
    Unix SMB/CIFS implementation.
    Test validity of smb.conf
    Copyright (C) Karl Auer 1993, 1994-1998
 
    Extensively modified by Andrew Tridgell, 1995
    Converted to popt by Jelmer Vernooij (jelmer@nl.linux.org), 2002
-   
+
    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 3 of the License, or
    (at your option) any later version.
-   
+
    This program is distributed in the hope that it will be useful,
    but WITHOUT ANY WARRANTY; without even the implied warranty of
    MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
    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, see <http://www.gnu.org/licenses/>.
 */
@@ -36,6 +36,8 @@
 #include "popt_common.h"
 #include "lib/param/loadparm.h"
 
+#include <regex.h>
+
 /*******************************************************************
  Check if a directory exists.
 ********************************************************************/
@@ -57,6 +59,147 @@ static bool directory_exist_stat(const char *dname,SMB_STRUCT_STAT *st)
        return ret;
 }
 
+struct idmap_config {
+       const char *domain_name;
+       const char *backend;
+       uint32_t high;
+       uint32_t low;
+};
+
+struct idmap_domains {
+       struct idmap_config *c;
+       uint32_t count;
+       uint32_t size;
+};
+
+static bool lp_scan_idmap_found_domain(const char *string,
+                                      regmatch_t matches[],
+                                      void *private_data)
+{
+       bool ok = false;
+
+       if (matches[1].rm_so == -1) {
+               fprintf(stderr, "Found match, but no name - invalid idmap config");
+               return false;
+       }
+       if (matches[1].rm_eo <= matches[1].rm_so) {
+               fprintf(stderr, "Invalid match - invalid idmap config");
+               return false;
+       }
+
+       {
+               struct idmap_domains *d = private_data;
+               struct idmap_config *c = &d->c[d->count];
+               regoff_t len = matches[1].rm_eo - matches[1].rm_so;
+               char domname[len + 1];
+
+               if (d->count >= d->size) {
+                       return false;
+               }
+
+               memcpy(domname, string + matches[1].rm_so, len);
+               domname[len] = '\0';
+
+               c->domain_name = talloc_strdup_upper(d->c, domname);
+               if (c->domain_name == NULL) {
+                       return false;
+               }
+               c->backend = talloc_strdup(d->c, lp_idmap_backend(domname));
+               if (c->backend == NULL) {
+                       return false;
+               }
+
+               if (lp_server_role() != ROLE_ACTIVE_DIRECTORY_DC) {
+                       ok = lp_idmap_range(domname, &c->low, &c->high);
+                       if (!ok) {
+                               fprintf(stderr,
+                                       "ERROR: Invalid idmap range for domain "
+                                       "%s!\n\n",
+                                       c->domain_name);
+                               return false;
+                       }
+               }
+
+               d->count++;
+       }
+
+       return false; /* Keep scanning */
+}
+
+static bool do_idmap_check(void)
+{
+       struct idmap_domains *d;
+       uint32_t i;
+       bool ok = false;
+       int rc;
+
+       d = talloc_zero(talloc_tos(), struct idmap_domains);
+       if (d == NULL) {
+               return false;
+       }
+       d->count = 0;
+       d->size = 32;
+
+       d->c = talloc_array(d, struct idmap_config, d->size);
+       if (d->c == NULL) {
+               goto done;
+       }
+
+       rc = lp_wi_scan_global_parametrics("idmapconfig\\(.*\\):backend",
+                                          2,
+                                          lp_scan_idmap_found_domain,
+                                          d);
+       if (rc != 0) {
+               fprintf(stderr,
+                       "FATAL: wi_scan_global_parametrics failed: %d",
+                       rc);
+       }
+
+       for (i = 0; i < d->count; i++) {
+               struct idmap_config *c = &d->c[i];
+               uint32_t j;
+
+               for (j = 0; j < d->count && j != i; j++) {
+                       struct idmap_config *x = &d->c[j];
+
+                       if ((c->low >= x->low && c->low <= x->high) ||
+                           (c->high >= x->low && c->high <= x->high)) {
+                               /* Allow overlapping ranges for idmap_ad */
+                               ok = strequal(c->backend, x->backend);
+                               if (ok) {
+                                       ok = strequal(c->backend, "ad");
+                                       if (ok) {
+                                               fprintf(stderr,
+                                                       "NOTE: The idmap_ad "
+                                                       "range for the domain "
+                                                       "%s overlaps with the "
+                                                       "range of %s.\n\n",
+                                                       c->domain_name,
+                                                       x->domain_name);
+                                               continue;
+                                       }
+                               }
+
+                               fprintf(stderr,
+                                       "ERROR: The idmap range for the domain "
+                                       "%s (%s) overlaps with the range of "
+                                       "%s (%s)!\n\n",
+                                       c->domain_name,
+                                       c->backend,
+                                       x->domain_name,
+                                       x->backend);
+                               ok = false;
+                               goto done;
+                       }
+               }
+       }
+
+       ok = true;
+done:
+       TALLOC_FREE(d);
+       return ok;
+}
+
 /***********************************************
  Here we do a set of 'hard coded' checks for bad
  configuration settings.
@@ -68,7 +211,7 @@ static int do_global_checks(void)
        SMB_STRUCT_STAT st;
        const char *socket_options;
 
-       if (lp_security() >= SEC_DOMAIN && !lp_encrypted_passwords()) {
+       if (lp_security() >= SEC_DOMAIN && !lp_encrypt_passwords()) {
                fprintf(stderr, "ERROR: in 'security=domain' mode the "
                                "'encrypt passwords' parameter must always be "
                                "set to 'true'.\n\n");
@@ -88,39 +231,56 @@ static int do_global_checks(void)
                                "must differ.\n\n");
        }
 
-       if (!directory_exist_stat(lp_lockdir(), &st)) {
+       if (lp_client_ipc_signing() == SMB_SIGNING_IF_REQUIRED
+        || lp_client_ipc_signing() == SMB_SIGNING_OFF) {
+               fprintf(stderr, "WARNING: The 'client ipc signing' value "
+                       "%s SMB signing is not used when contacting a "
+                       "domain controller or other server. "
+                       "This setting is not recommended; please be "
+                       "aware of the security implications when using "
+                       "this configuration setting.\n\n",
+                       lp_client_ipc_signing() == SMB_SIGNING_OFF ?
+                       "ensures" : "may mean");
+       }
+
+       if (strlen(lp_netbios_name()) > 15) {
+               fprintf(stderr, "WARNING: The 'netbios name' is too long "
+                               "(max. 15 chars).\n\n");
+       }
+
+       if (!directory_exist_stat(lp_lock_directory(), &st)) {
                fprintf(stderr, "ERROR: lock directory %s does not exist\n\n",
-                      lp_lockdir());
+                      lp_lock_directory());
                ret = 1;
        } else if ((st.st_ex_mode & 0777) != 0755) {
                fprintf(stderr, "WARNING: lock directory %s should have "
                                "permissions 0755 for browsing to work\n\n",
-                      lp_lockdir());
+                      lp_lock_directory());
        }
 
-       if (!directory_exist_stat(lp_statedir(), &st)) {
+       if (!directory_exist_stat(lp_state_directory(), &st)) {
                fprintf(stderr, "ERROR: state directory %s does not exist\n\n",
-                      lp_statedir());
+                      lp_state_directory());
                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\n",
-                      lp_statedir());
+                      lp_state_directory());
        }
 
-       if (!directory_exist_stat(lp_cachedir(), &st)) {
+       if (!directory_exist_stat(lp_cache_directory(), &st)) {
                fprintf(stderr, "ERROR: cache directory %s does not exist\n\n",
-                      lp_cachedir());
+                      lp_cache_directory());
                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\n",
-                      lp_cachedir());
+                      lp_cache_directory());
        }
 
-       if (!directory_exist_stat(lp_piddir(), &st)) {
+       if (!directory_exist_stat(lp_pid_directory(), &st)) {
                fprintf(stderr, "ERROR: pid directory %s does not exist\n\n",
-                      lp_piddir());
+                      lp_pid_directory());
                ret = 1;
        }
 
@@ -165,7 +325,7 @@ static int do_global_checks(void)
         * Password server sanity checks.
         */
 
-       if((lp_security() >= SEC_DOMAIN) && !*lp_passwordserver()) {
+       if((lp_security() >= SEC_DOMAIN) && !*lp_password_server()) {
                const char *sec_setting;
                if(lp_security() == SEC_DOMAIN)
                        sec_setting = "domain";
@@ -181,7 +341,7 @@ static int do_global_checks(void)
                ret = 1;
        }
 
-       if((lp_security() >= SEC_DOMAIN) && (strcmp(lp_passwordserver(), "*") != 0)) {
+       if((lp_security() >= SEC_DOMAIN) && (strcmp(lp_password_server(), "*") != 0)) {
                const char *sec_setting;
                if(lp_security() == SEC_DOMAIN)
                        sec_setting = "domain";
@@ -272,7 +432,7 @@ static int do_global_checks(void)
                 * been written to expect the old password.
                 */
 
-               if(lp_encrypted_passwords()) {
+               if(lp_encrypt_passwords()) {
                        if(strstr_m( lp_passwd_chat(talloc_tos()), "%o")!=NULL) {
                                fprintf(stderr,
                                        "ERROR: the 'passwd chat' script [%s] "
@@ -308,6 +468,37 @@ static int do_global_checks(void)
                fprintf(stderr, "'algorithmic rid base' must be even.\n\n");
        }
 
+       if (lp_server_role() != ROLE_STANDALONE) {
+               const char *default_backends[] = {
+                       "tdb", "tdb2", "ldap", "autorid", "hash"
+               };
+               const char *idmap_backend;
+               bool valid_backend = false;
+               uint32_t i;
+               bool ok;
+
+               idmap_backend = lp_idmap_default_backend();
+
+               for (i = 0; i < ARRAY_SIZE(default_backends); i++) {
+                       ok = strequal(idmap_backend, default_backends[i]);
+                       if (ok) {
+                               valid_backend = true;
+                       }
+               }
+
+               if (!valid_backend) {
+                       ret = 1;
+                       fprintf(stderr, "ERROR: Do not use the '%s' backend "
+                                       "as the default idmap backend!\n\n",
+                                       idmap_backend);
+               }
+
+               ok = do_idmap_check();
+               if (!ok) {
+                       ret = 1;
+               }
+       }
+
 #ifndef HAVE_DLOPEN
        if (lp_preload_modules()) {
                fprintf(stderr, "WARNING: 'preload modules = ' set while loading "
@@ -319,7 +510,7 @@ static int do_global_checks(void)
                fprintf(stderr, "ERROR: passdb backend must have a value or be "
                                "left out\n\n");
        }
-       
+
        if (lp_os_level() > 255) {
                fprintf(stderr, "WARNING: Maximum value for 'os level' is "
                                "255!\n\n");
@@ -331,16 +522,20 @@ static int do_global_checks(void)
        }
 
        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);
+       const char **deny_list = lp_hosts_deny(s);
+       const char **allow_list = lp_hosts_allow(s);
+       const char **vfs_objects = NULL;
        int i;
+       static bool uses_fruit;
+       static bool doesnt_use_fruit;
+       static bool fruit_mix_warned;
 
        if(deny_list) {
                for (i=0; deny_list[i]; i++) {
@@ -415,12 +610,26 @@ static void do_per_share_checks(int s)
                        "excludes octal 010 (S_IXGRP).\n\n",
                        lp_servicename(talloc_tos(), s));
        }
-       if (lp_printing(s) == PRINT_CUPS && *(lp_printcommand(talloc_tos(), s)) != '\0') {
+       if (lp_printing(s) == PRINT_CUPS && *(lp_print_command(talloc_tos(), s)) != '\0') {
                fprintf(stderr,
                        "Warning: Service %s defines a print command, but "
                        "parameter is ignored when using CUPS libraries.\n\n",
                        lp_servicename(talloc_tos(), s));
        }
+
+       vfs_objects = lp_vfs_objects(s);
+       if (vfs_objects && str_list_check(vfs_objects, "fruit")) {
+               uses_fruit = true;
+       } else {
+               doesnt_use_fruit = true;
+       }
+
+       if (uses_fruit && doesnt_use_fruit && !fruit_mix_warned) {
+               fruit_mix_warned = true;
+               fprintf(stderr,
+                       "WARNING: some services use vfs_fruit, others don't. Mounting them "
+                       "in conjunction on OS X clients results in undefined behaviour.\n\n");
+       }
 }
 
  int main(int argc, const char *argv[])
@@ -454,7 +663,7 @@ static void do_per_share_checks(int s)
 
        TALLOC_CTX *frame = talloc_stackframe();
 
-       load_case_tables();
+       smb_init_locale();
        /*
         * Set the default debug level to 2.
         * Allow it to be overridden by the command line,
@@ -462,7 +671,7 @@ static void do_per_share_checks(int s)
         */
        lp_set_cmdline("log level", "2");
 
-       pc = poptGetContext(NULL, argc, argv, long_options, 
+       pc = poptGetContext(NULL, argc, argv, long_options,
                            POPT_CONTEXT_KEEP_FIRST);
        poptSetOtherOptionHelp(pc, "[OPTION...] <config-file> [host-name] [host-ip]");
 
@@ -475,7 +684,7 @@ static void do_per_share_checks(int s)
 
        setup_logging(poptGetArg(pc), DEBUG_STDERR);
 
-       if (poptPeekArg(pc)) 
+       if (poptPeekArg(pc))
                config_file = poptGetArg(pc);
 
        cname = poptGetArg(pc);
@@ -491,7 +700,7 @@ static void do_per_share_checks(int s)
 
        fprintf(stderr,"Load smb config files from %s\n",config_file);
 
-       if (!lp_load_with_registry_shares(config_file,False,True,False,True)) {
+       if (!lp_load_with_registry_shares(config_file)) {
                fprintf(stderr,"Error loading services.\n");
                ret = 1;
                goto done;
@@ -569,8 +778,8 @@ static void do_per_share_checks(int s)
                /* this is totally ugly, a real `quick' hack */
                for (s=0;s<1000;s++) {
                        if (VALID_SNUM(s)) {
-                               if (allow_access(lp_hostsdeny(-1), lp_hostsallow(-1), cname, caddr)
-                                   && allow_access(lp_hostsdeny(s), lp_hostsallow(s), cname, caddr)) {
+                               if (allow_access(lp_hosts_deny(-1), lp_hosts_allow(-1), cname, caddr)
+                                   && allow_access(lp_hosts_deny(s), lp_hosts_allow(s), cname, caddr)) {
                                        fprintf(stderr,"Allow connection from %s (%s) to %s\n",
                                                   cname,caddr,lp_servicename(talloc_tos(), s));
                                } else {