svn commit: samba r21394 - in branches: SAMBA_3_0/source/nsswitch SAMBA_3_0_25/source/nsswitch

[gd at samba.org]

Author: gd
Date: 2007-02-16 17:25:54 +0000 (Fri, 16 Feb 2007)
New Revision: 21394

WebSVN: http://websvn.samba.org/cgi-bin/viewcvs.cgi?view=rev&root=samba&rev=21394

Log:
Prevent nscd crash due to potential NULL pointer dereference in
_nss_winbind_initgroups_dyn() on an empty group list.

Guenther

Modified:
   branches/SAMBA_3_0/source/nsswitch/winbind_nss_linux.c
   branches/SAMBA_3_0_25/source/nsswitch/winbind_nss_linux.c


Changeset:
Modified: branches/SAMBA_3_0/source/nsswitch/winbind_nss_linux.c
===================================================================
--- branches/SAMBA_3_0/source/nsswitch/winbind_nss_linux.c	2007-02-16 16:59:28 UTC (rev 21393)
+++ branches/SAMBA_3_0/source/nsswitch/winbind_nss_linux.c	2007-02-16 17:25:54 UTC (rev 21394)
@@ -948,6 +948,10 @@
 				"and %d gids\n", getpid(),
 				user, num_gids);
 #endif
+		if (gid_list == NULL) {
+			ret = NSS_STATUS_NOTFOUND;
+			goto done;
+		}
 
 		/* Copy group list to client */
 

Modified: branches/SAMBA_3_0_25/source/nsswitch/winbind_nss_linux.c
===================================================================
--- branches/SAMBA_3_0_25/source/nsswitch/winbind_nss_linux.c	2007-02-16 16:59:28 UTC (rev 21393)
+++ branches/SAMBA_3_0_25/source/nsswitch/winbind_nss_linux.c	2007-02-16 17:25:54 UTC (rev 21394)
@@ -948,6 +948,10 @@
 				"and %d gids\n", getpid(),
 				user, num_gids);
 #endif
+		if (gid_list == NULL) {
+			ret = NSS_STATUS_NOTFOUND;
+			goto done;
+		}
 
 		/* Copy group list to client */