summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorGünther Deschner <gd@samba.org>2009-06-07 11:23:09 +0200
committerKarolin Seeger <kseeger@samba.org>2009-06-15 10:12:19 +0200
commit51f6f4343568b19e4c6ba2a3f919931b26228531 (patch)
tree41e7c762686b38928d41398e0e4b836ce56a9232
parentd0690a1f34658f0c08307d5287eba5ca2d2da9d5 (diff)
downloadsamba-51f6f4343568b19e4c6ba2a3f919931b26228531.tar.gz
samba-51f6f4343568b19e4c6ba2a3f919931b26228531.tar.xz
samba-51f6f4343568b19e4c6ba2a3f919931b26228531.zip
s3-groupdb: fix enum_aliasmem in ldb branch.
It is totally valid to have an alias with no members. This fixes bug #6465. Tridge, please check. Found by RPC-SAMR torture test. Guenther (cherry picked from commit d27c4346397b828e3d7d01473f174577440d950a)
-rw-r--r--source/groupdb/mapping_ldb.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/source/groupdb/mapping_ldb.c b/source/groupdb/mapping_ldb.c
index a69d3064057..5f79aaa9e83 100644
--- a/source/groupdb/mapping_ldb.c
+++ b/source/groupdb/mapping_ldb.c
@@ -543,7 +543,7 @@ static NTSTATUS enum_aliasmem(const DOM_SID *alias, DOM_SID **sids, size_t *num)
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++) {