Pass a talloc_ctx to pdb_enum_aliasmem

[ira/wip.git] / source3 / groupdb / mapping_ldb.c

diff --git a/source3/groupdb/mapping_ldb.c b/source3/groupdb/mapping_ldb.c
index e16476715df1aa79f0a320f8c5258e547751bd81..89966e1986e779c506e83a842f3577e682f42ac6 100644 (file)
--- a/source3/groupdb/mapping_ldb.c
+++ b/source3/groupdb/mapping_ldb.c
@@ -225,7 +225,7 @@ static bool get_group_map_from_sid(DOM_SID sid, GROUP_MAP *map)
        }
 
        ret = ldb_search(ldb, dn, &res, dn, LDB_SCOPE_BASE, NULL, NULL);
-       if (ret != LDB_SUCCESS || res == NULL || res->count != 1) {
+       if (ret != LDB_SUCCESS || res->count != 1) {
                goto failed;
        }
 
@@ -251,7 +251,7 @@ static bool get_group_map_from_gid(gid_t gid, GROUP_MAP *map)
        ret = ldb_search(ldb, talloc_tos(), &res, NULL, LDB_SCOPE_SUBTREE,
                         NULL, "(&(gidNumber=%u)(objectClass=groupMap))",
                         (unsigned)gid);
-       if (ret != LDB_SUCCESS || res == NULL || res->count != 1) {
+       if (ret != LDB_SUCCESS || res->count != 1) {
                goto failed;
        }
 
@@ -276,7 +276,7 @@ static bool get_group_map_from_ntname(const char *name, GROUP_MAP *map)
 
        ret = ldb_search(ldb, talloc_tos(), &res, NULL, LDB_SCOPE_SUBTREE,
                         NULL, "(&(ntName=%s)(objectClass=groupMap))", name);
-       if (ret != LDB_SUCCESS || res == NULL || res->count != 1) {
+       if (ret != LDB_SUCCESS || res->count != 1) {
                goto failed;
        }
 
@@ -341,7 +341,7 @@ static bool enum_group_mapping(const DOM_SID *domsid, enum lsa_SidType sid_name_
                                 sid_name_use);
        }
 
-       if (ret != LDB_SUCCESS || res == NULL) goto failed;
+       if (ret != LDB_SUCCESS) goto failed;
 
        (*pp_rmap) = NULL;
        *p_num_entries = 0;
@@ -390,7 +390,7 @@ static NTSTATUS one_alias_membership(const DOM_SID *member,
        ret = ldb_search(ldb, talloc_tos(), &res, NULL, LDB_SCOPE_SUBTREE,
                         attrs, "(&(member=%s)(objectClass=groupMap))",
                         string_sid);
-       if (ret != LDB_SUCCESS || res == NULL) {
+       if (ret != LDB_SUCCESS) {
                status = NT_STATUS_INTERNAL_DB_CORRUPTION;
                goto failed;
        }
@@ -488,7 +488,8 @@ static NTSTATUS del_aliasmem(const DOM_SID *alias, const DOM_SID *member)
 /*
   enumerate sids that have the given alias set in member
 */
-static NTSTATUS enum_aliasmem(const DOM_SID *alias, DOM_SID **sids, size_t *num)
+static NTSTATUS enum_aliasmem(const DOM_SID *alias, TALLOC_CTX *mem_ctx,
+                             DOM_SID **sids, size_t *num)
 {
        const char *attrs[] = {
                "member",
@@ -509,8 +510,8 @@ static NTSTATUS enum_aliasmem(const DOM_SID *alias, DOM_SID **sids, size_t *num)
        }
 
        ret = ldb_search(ldb, ldb, &res, dn, LDB_SCOPE_BASE, attrs, NULL);
-       talloc_steal(dn, res);
-       if (ret == LDB_SUCCESS && res && res->count == 0) {
+       if (ret == LDB_SUCCESS && res->count == 0) {
+               talloc_free(res);
                talloc_free(dn);
                return NT_STATUS_OK;
        }
@@ -519,16 +520,17 @@ static NTSTATUS enum_aliasmem(const DOM_SID *alias, DOM_SID **sids, size_t *num)
                return NT_STATUS_INTERNAL_DB_CORRUPTION;
        }
 
+       talloc_steal(dn, res);
        el = ldb_msg_find_element(res->msgs[0], "member");
        if (el == NULL) {
                talloc_free(dn);
-               return NT_STATUS_INTERNAL_DB_CORRUPTION;
+               return NT_STATUS_OK;
        }
        
        for (i=0;i<el->num_values;i++) {
                DOM_SID sid;
                string_to_sid(&sid, (const char *)el->values[i].data);
-               status = add_sid_to_array_unique(NULL, &sid, sids, num);
+               status = add_sid_to_array_unique(mem_ctx, &sid, sids, num);
                if (!NT_STATUS_IS_OK(status)) {
                        goto done;
                }