From b78d44fe89f3a2f6635b46cbcfae28069c9cff55 Mon Sep 17 00:00:00 2001 From: Michael Adam Date: Sun, 20 Jul 2014 12:12:16 +0200 Subject: [PATCH] s3:idmap: only check the range values if a range setting has been found. Otherwise, the check is superfluous since high and low values are initialized to 0. Signed-off-by: Michael Adam Reviewed-by: Volker Lendecke --- source3/winbindd/idmap.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/source3/winbindd/idmap.c b/source3/winbindd/idmap.c index e62f477764d..dc972fea00e 100644 --- a/source3/winbindd/idmap.c +++ b/source3/winbindd/idmap.c @@ -238,9 +238,7 @@ static struct idmap_domain *idmap_init_domain(TALLOC_CTX *mem_ctx, if (check_range) { goto fail; } - } - - if (result->low_id > result->high_id) { + } else if (result->low_id > result->high_id) { DEBUG(1, ("Error: invalid idmap range detected: %lu - %lu\n", (unsigned long)result->low_id, (unsigned long)result->high_id)); -- 2.11.4.GIT