d/freebsd-nq
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Fix problem where initgroups would silently truncate groups with

Browse Source 
more than NGROUP elements without providing the opportunity to
setgroups to fail and correctly return error and set errno.

MFC after:	2 weeks
This commit is contained in:
Diomidis Spinellis 2003-11-19 15:51:26 +00:00
parent 6006b8f4c6
commit 434c74760f
1 changed files with 7 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
lib/libc/gen/initgroups.c
View File

@ -50,9 +50,14 @@ initgroups(uname, agroup)
const char *uname; const char *uname;
gid_t agroup; gid_t agroup;
{ {
int groups[NGROUPS], ngroups; int ngroups;
/*
* Provide space for one group more than NGROUPS to allow
* setgroups to fail and set errno.
*/
gid_t groups[NGROUPS + 1];
ngroups = NGROUPS; ngroups = NGROUPS + 1;
getgrouplist(uname, agroup, groups, &ngroups); getgrouplist(uname, agroup, groups, &ngroups);
return (setgroups(ngroups, groups)); return (setgroups(ngroups, groups));
} }