[SCM] Samba Shared Repository - branch master updated
Jim McDonough
jmcd at samba.org
Wed Dec 30 13:36:24 MST 2009
The branch, master has been updated
via 3a271a8... Prevent NULL dereference if group has no members
from 1aed373... s4:lib/registry/util.c - Reintroduce "FIXME"s
http://gitweb.samba.org/?p=samba.git;a=shortlog;h=master
- Log -----------------------------------------------------------------
commit 3a271a89b5e6d23089f9391f0cfd42f6631c925a
Author: Jim McDonough <jmcd at samba.org>
Date: Wed Dec 30 15:04:55 2009 -0500
Prevent NULL dereference if group has no members
-----------------------------------------------------------------------
Summary of changes:
source3/winbindd/winbindd_rpc.c | 8 ++++----
1 files changed, 4 insertions(+), 4 deletions(-)
Changeset truncated at 500 lines:
diff --git a/source3/winbindd/winbindd_rpc.c b/source3/winbindd/winbindd_rpc.c
index ffe488a..e700376 100644
--- a/source3/winbindd/winbindd_rpc.c
+++ b/source3/winbindd/winbindd_rpc.c
@@ -787,16 +787,16 @@ static NTSTATUS lookup_groupmem(struct winbindd_domain *domain,
if (!NT_STATUS_IS_OK(result))
return result;
- *num_names = rids->count;
- rid_mem = rids->rids;
-
- if (!*num_names) {
+ if (!rids || !rids->count) {
names = NULL;
name_types = NULL;
sid_mem = NULL;
return NT_STATUS_OK;
}
+ *num_names = rids->count;
+ rid_mem = rids->rids;
+
/* Step #2: Convert list of rids into list of usernames. Do this
in bunches of ~1000 to avoid crashing NT4. It looks like there
is a buffer overflow or something like that lurking around
--
Samba Shared Repository
More information about the samba-cvs
mailing list