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.