diff options
author | Jeremy Allison <jra@samba.org> | 2009-05-21 18:37:36 -0700 |
---|---|---|
committer | Karolin Seeger <kseeger@samba.org> | 2009-05-26 09:39:47 +0200 |
commit | 74fa6aa409c5539e22d28d2f32fab9fab2f0236c (patch) | |
tree | 709d849768cf387ac2c28d7e8fe6fa4fd976a9f3 /source3 | |
parent | da7372fdb0d2c29619dbcb1c180317c0f2bd834a (diff) | |
download | samba-74fa6aa409c5539e22d28d2f32fab9fab2f0236c.tar.gz |
Revert the last two commits (fix for #6386). The actual problem
was a bug in ldb in 3.2 which could return a freed pointer on
ret != LDAP_SUCCESS. The main thing we must ensure is that we
never talloc_steal until we know LDAP_SUCCESS was returned.
Jeremy.
(cherry picked from commit f3c3ee0f5dc6266f58e96606b73f55b812fe5171)
Diffstat (limited to 'source3')
-rw-r--r-- | source3/groupdb/mapping_ldb.c | 16 |
1 files changed, 9 insertions, 7 deletions
diff --git a/source3/groupdb/mapping_ldb.c b/source3/groupdb/mapping_ldb.c index 19804858a68..2bb25b2e410 100644 --- 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; } @@ -509,16 +509,18 @@ 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_steal(dn, res); talloc_free(dn); return NT_STATUS_OK; } if (ret != LDB_SUCCESS) { + talloc_steal(dn, res); talloc_free(dn); 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); |