s3:idmap: don't log missing range config if range checking not requested
authorMichael Adam <obnox@samba.org>
Sun, 20 Jul 2014 09:49:37 +0000 (11:49 +0200)
committerVolker Lendecke <vl@samba.org>
Fri, 25 Jul 2014 09:52:10 +0000 (11:52 +0200)
idmap_init_domain() is called with check_range == false from
idmap_passdb_domain(). In this case, we usually don't have an
idmap range at all, and we don't want to level 1 debug
messages complaining about the fact are irritating at least.

This patch removes the debug in the case of check_range == false.

BUG: https://bugzilla.samba.org/show_bug.cgi?id=10737

Signed-off-by: Michael Adam <obnox@samba.org>
Reviewed-by: Volker Lendecke <vl@samba.org>
source3/winbindd/idmap.c

index 674f54ced1cd7d1395e0eb48e135d86bbc47783c..33b397ce661001f1575b4c1dfc5c90d2783be2c1 100644 (file)
@@ -198,9 +198,9 @@ static struct idmap_domain *idmap_init_domain(TALLOC_CTX *mem_ctx,
 
        range = lp_parm_const_string(-1, config_option, "range", NULL);
        if (range == NULL) {
-               DEBUG(1, ("idmap range not specified for domain %s\n",
-                         result->name));
                if (check_range) {
+                       DEBUG(1, ("idmap range not specified for domain %s\n",
+                                 result->name));
                        goto fail;
                }
        } else if (sscanf(range, "%u - %u", &result->low_id,