summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorJeremy Allison <jra@samba.org>2009-05-21 18:00:54 -0700
committerJeremy Allison <jra@samba.org>2009-05-21 18:00:54 -0700
commit87504b27d80107b728940e1ecdbd1d33e761779b (patch)
tree5eb7170dd44fab3da1b0ce0eee6ca965b6d95fbf
parentd4d06a4ef921e8fa543b86de002829ba0e7f77d3 (diff)
downloadsamba-87504b27d80107b728940e1ecdbd1d33e761779b.tar.gz
samba-87504b27d80107b728940e1ecdbd1d33e761779b.tar.bz2
samba-87504b27d80107b728940e1ecdbd1d33e761779b.zip
Ensure all possible uses of indirection through res are checked after
an ldb_search. Jeremy.
-rw-r--r--source3/groupdb/mapping_ldb.c12
1 files changed, 6 insertions, 6 deletions
diff --git a/source3/groupdb/mapping_ldb.c b/source3/groupdb/mapping_ldb.c
index b95ba0a9f9..e16476715d 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->count != 1) {
+ if (ret != LDB_SUCCESS || res == NULL || 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->count != 1) {
+ if (ret != LDB_SUCCESS || res == NULL || 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 && res->count != 1)) {
+ if (ret != LDB_SUCCESS || res == NULL || 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) goto failed;
+ if (ret != LDB_SUCCESS || res == NULL) 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) {
+ if (ret != LDB_SUCCESS || res == NULL) {
status = NT_STATUS_INTERNAL_DB_CORRUPTION;
goto failed;
}
@@ -510,7 +510,7 @@ 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->count == 0) {
+ if (ret == LDB_SUCCESS && res && res->count == 0) {
talloc_free(dn);
return NT_STATUS_OK;
}