CVS commit: src/usr.bin/su

[Christos Zoulas <christos%netbsd.org@localhost>]

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.