Subject: CVS commit: src/usr.bin/su
To: None <source-changes@NetBSD.org>
From: Christos Zoulas <christos@netbsd.org>
List: source-changes
Date: 10/27/2007 15:33:25
Module Name: src
Committed By: christos
Date: Sat Oct 27 15:33:25 UTC 2007
Modified Files:
src/usr.bin/su: grutil.c
Log Message:
From Anon Ymous:
- Fix a memory leak in newgrp(). (Noted by ginsbach@.)
- If the new and old groups are the same, addgrp() does nothing so
exit early. (Noted by ginsbach@.)
- In newgrp(), if the new group isn't found in the in-core, check the
group membership list anyway as the in-core list may be full or
/etc/groups may have been edited since login. (Requested by
ginsbach@.)
- Deal with a full in-core group[] array in addgid() rather than
silently ignoring it and ending up with a group[] array that is
inconsistent with the current group.
- More explicit error handling in addgroup().
To generate a diff of this commit:
cvs rdiff -r1.2 -r1.3 src/usr.bin/su/grutil.c
Please note that diffs are not public domain; they are subject to the
copyright notices on the relevant files.