idmap-autorid: Remove an else branch

Signed-off-by: Christian Ambach <christian.ambach@de.ibm.com>

diff --git a/source3/winbindd/idmap_autorid.c b/source3/winbindd/idmap_autorid.c
index 9fd7f15095a0a731e7ea1b3c27a80280cc41901e..5747a0b0c832ff57d9af21aae40e514aafd4aa32 100644 (file)
--- 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;
 }