Subject: PR/31069 CVS commit: [netbsd-3] src/usr.bin/id
To: None <gnats-admin@netbsd.org, netbsd-bugs@netbsd.org>
From: Matthias Scheler <tron@netbsd.org>
List: netbsd-bugs
Date: 09/12/2005 18:40:02
The following reply was made to PR bin/31069; it has been noted by GNATS.

From: Matthias Scheler <tron@netbsd.org>
To: gnats-bugs@netbsd.org
Cc: 
Subject: PR/31069 CVS commit: [netbsd-3] src/usr.bin/id
Date: Mon, 12 Sep 2005 18:39:22 +0000 (UTC)

 Module Name:	src
 Committed By:	tron
 Date:		Mon Sep 12 18:39:22 UTC 2005
 
 Modified Files:
 	src/usr.bin/id [netbsd-3]: id.c
 
 Log Message:
 Pull up following revision(s) (requested by drochner in ticket #772):
 	usr.bin/id/id.c: revision 1.25
 id(1) didn't check the return value of getgrouplist(3) and used memory
 outside the group array in the case that a user is member of more than
 _SC_NGROUPS_MAX groups.
 (This is probably also the problem behind PR bin/31069 by Zafer Aydogan.)
 So check the return value and retry with sufficiently allocated memory
 in case the initial _SC_NGROUPS_MAX groups are not enough.
 
 
 To generate a diff of this commit:
 cvs rdiff -r1.24 -r1.24.2.1 src/usr.bin/id/id.c
 
 Please note that diffs are not public domain; they are subject to the
 copyright notices on the relevant files.