s3:utils: Remove pointless if-clause for remote_machine
[metze/samba/wip.git] / source3 / utils / testparm.c
index a92fc3a8d8c5d6a293388a632dbb46b6cf8b37f7..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.
@@ -88,6 +231,23 @@ static int do_global_checks(void)
                                "must differ.\n\n");
        }
 
+       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_lock_directory());
@@ -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,7 +522,7 @@ static int do_global_checks(void)
        }
 
        return ret;
-}   
+}
 
 /**
  * per-share logic tests
@@ -340,7 +531,11 @@ static void do_per_share_checks(int 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++) {
@@ -421,6 +616,20 @@ static void do_per_share_checks(int s)
                        "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[])
@@ -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, True)) {
+       if (!lp_load_with_registry_shares(config_file)) {
                fprintf(stderr,"Error loading services.\n");
                ret = 1;
                goto done;