summaryrefslogtreecommitdiffstats
path: root/source3/winbindd
diff options
context:
space:
mode:
authorMichael Adam <obnox@samba.org>2014-07-20 12:12:16 +0200
committerVolker Lendecke <vl@samba.org>2014-07-25 11:52:10 +0200
commitb78d44fe89f3a2f6635b46cbcfae28069c9cff55 (patch)
treeabf96875a1342c33fc9e99c019743f48f815ab82 /source3/winbindd
parentf3549172083641d0e275a6312edf4adc6b740b98 (diff)
downloadsamba-b78d44fe89f3a2f6635b46cbcfae28069c9cff55.tar.gz
samba-b78d44fe89f3a2f6635b46cbcfae28069c9cff55.tar.xz
samba-b78d44fe89f3a2f6635b46cbcfae28069c9cff55.zip
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 <obnox@samba.org> Reviewed-by: Volker Lendecke <vl@samba.org>
Diffstat (limited to 'source3/winbindd')
-rw-r--r--source3/winbindd/idmap.c4
1 files changed, 1 insertions, 3 deletions
diff --git a/source3/winbindd/idmap.c b/source3/winbindd/idmap.c
index e62f477764..dc972fea00 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));