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.