Attachment 5112 Details for Bug 7014 – backport to 3.4

backport to 3.4

0001-Prevent-NULL-dereference-if-group-has-no-members.patch (text/plain), 1.10 KB, created by Jim McDonough on 2009-12-30 15:02:51 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Jim McDonough

Created: 2009-12-30 15:02:51 UTC

Size: 1.10 KB

patch

obsolete

>From e9ce354eeed8a4830ef5596edc1a1b88d41e96e1 Mon Sep 17 00:00:00 2001
>From: Jim McDonough <jmcd@samba.org>
>Date: Wed, 30 Dec 2009 15:04:55 -0500
>Subject: [PATCH] Prevent NULL dereference if group has no members
>
>---
> source3/winbindd/winbindd_rpc.c |    8 ++++----
> 1 files changed, 4 insertions(+), 4 deletions(-)
>
>diff --git a/source3/winbindd/winbindd_rpc.c b/source3/winbindd/winbindd_rpc.c
>index f176fb3..aa8a5c8 100644
>--- a/source3/winbindd/winbindd_rpc.c
>+++ b/source3/winbindd/winbindd_rpc.c
>@@ -789,16 +789,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
>-- 
>1.6.4.2
>

Actions: View

Attachments on bug 7014: 5112 | 5113