From 6ba85beeec7ab11baba548c7443e904d9ea31944 Mon Sep 17 00:00:00 2001 From: Volker Lendecke Date: Tue, 8 Mar 2011 21:45:02 +0100 Subject: idmap-autorid: Remove an else branch Signed-off-by: Christian Ambach --- source3/winbindd/idmap_autorid.c | 21 +++++++++++---------- 1 file changed, 11 insertions(+), 10 deletions(-) (limited to 'source3/winbindd') diff --git a/source3/winbindd/idmap_autorid.c b/source3/winbindd/idmap_autorid.c index 9fd7f15095..5747a0b0c8 100644 --- a/source3/winbindd/idmap_autorid.c +++ b/source3/winbindd/idmap_autorid.c @@ -168,19 +168,20 @@ static NTSTATUS idmap_autorid_id_to_sid(TALLOC_CTX * memctx, DEBUG(4, ("id %d belongs to range %d which does not have " "domain mapping, ignoring mapping request\n", map->xid.id, range)); - } else { - string_to_sid(&sid, (const char *)data.dptr); + return NT_STATUS_OK; + } - sid_compose(map->sid, &sid, - (map->xid.id - cfg->minvalue - - range * cfg->rangesize)); + string_to_sid(&sid, (const char *)data.dptr); - /* We **really** should have some way of validating - the SID exists and is the correct type here. But - that is a deficiency in the idmap_rid design. */ + sid_compose(map->sid, &sid, + (map->xid.id - cfg->minvalue - + range * cfg->rangesize)); - map->status = ID_MAPPED; - } + /* We **really** should have some way of validating + the SID exists and is the correct type here. But + that is a deficiency in the idmap_rid design. */ + + map->status = ID_MAPPED; return NT_STATUS_OK; } -- cgit