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)
committerKarolin Seeger <kseeger@samba.org>
Thu, 7 Aug 2014 14:43:09 +0000 (16:43 +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>
(cherry picked from commit 3c6ec8908a697ac95536b35d242ccd64e524a0a5)

source3/winbindd/idmap.c

index 97a34d4bddb34c8fef41b0b9d5da861be9322926..c3c88eae1385878c1e688d5834bb72ec21679d8e 100644 (file)
@@ -197,9 +197,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,