[SCM] Samba Shared Repository - branch v3-4-test updated -
release-4-0-0alpha7-449-ga1cf98a
Volker Lendecke
vlendec at samba.org
Tue Mar 17 20:19:29 GMT 2009
The branch, v3-4-test has been updated
via a1cf98a2458a7f7a08531832bd29da3962ddafc9 (commit)
from fa9d1f633796bff86e899b48e3ea8bb77492c10b (commit)
http://gitweb.samba.org/?p=samba.git;a=shortlog;h=v3-4-test
- Log -----------------------------------------------------------------
commit a1cf98a2458a7f7a08531832bd29da3962ddafc9
Author: Volker Lendecke <vl at samba.org>
Date: Mon Mar 16 13:17:04 2009 +0100
Fix #6130: Don't crash in winbindd_rpc lookup_groupmem() on unmapped members
Thanks to François Legal <devel at thom.fr.eu.org> for reporting this bug
-----------------------------------------------------------------------
Summary of changes:
source3/winbindd/winbindd_rpc.c | 15 ++++++++-------
1 files changed, 8 insertions(+), 7 deletions(-)
Changeset truncated at 500 lines:
diff --git a/source3/winbindd/winbindd_rpc.c b/source3/winbindd/winbindd_rpc.c
index 0070bde..2c0222e 100644
--- a/source3/winbindd/winbindd_rpc.c
+++ b/source3/winbindd/winbindd_rpc.c
@@ -857,14 +857,15 @@ static NTSTATUS lookup_groupmem(struct winbindd_domain *domain,
}
for (r=0; r<tmp_names.count; r++) {
- (*names)[i+r] = fill_domain_username_talloc(mem_ctx,
- domain->name,
- tmp_names.names[r].string,
- true);
- (*name_types)[i+r] = tmp_types.ids[r];
+ if (tmp_types.ids[r] == SID_NAME_UNKNOWN) {
+ continue;
+ }
+ (*names)[total_names] = fill_domain_username_talloc(
+ mem_ctx, domain->name,
+ tmp_names.names[r].string, true);
+ (*name_types)[total_names] = tmp_types.ids[r];
+ total_names += 1;
}
-
- total_names += tmp_names.count;
}
*num_names = total_names;
--
Samba Shared Repository
More information about the samba-cvs
mailing list