]> git.ipfire.org Git - thirdparty/shadow.git/commit
* lib/prototypes.h, configure.in, libmisc/Makefile.am,
authornekral-guest <nekral-guest@5a98b0ae-9ef6-0310-add3-de5d479b70d7>
Sun, 18 Nov 2007 23:15:26 +0000 (23:15 +0000)
committernekral-guest <nekral-guest@5a98b0ae-9ef6-0310-add3-de5d479b70d7>
Sun, 18 Nov 2007 23:15:26 +0000 (23:15 +0000)
commit9adfc136b6624c95270c066cc3e8c559f3daee0a
tree95734485cc335a33b7da85ca1dc7eab656411219
parentea63711c2cb7188fbb0ccd429da3972e59d992a1
* lib/prototypes.h, configure.in, libmisc/Makefile.am,
  libmisc/xgetXXbyYY.c, libmisc/xgetpwnam.c, libmisc/xgetpwuid.c,
  libmisc/xgetgrnam.c, libmisc/xgetgrgid.c, libmisc/xgetspnam.c:
  Added functions xgetpwnam(), xgetpwuid(), xgetgrnam(),
  xgetgrgid(), and xgetspnam(). They allocate memory for the
  returned structure and are more robust to successive calls. They
  are implemented with the libc's getxxyyy_r() functions if
  available.
* libmisc/limits.c, libmisc/entry.c, libmisc/chowntty.c,
  libmisc/addgrps.c, libmisc/myname.c, libmisc/rlogin.c,
  libmisc/pwdcheck.c, src/newgrp.c, src/login_nopam.c,
  src/userdel.c, src/lastlog.c, src/grpck.c, src/gpasswd.c,
  src/newusers.c, src/chpasswd.c, src/chfn.c, src/groupmems.c,
  src/usermod.c, src/expiry.c, src/groupdel.c, src/chgpasswd.c,
  src/su.c, src/useradd.c, src/groupmod.c, src/passwd.c, src/pwck.c,
  src/groupadd.c, src/chage.c, src/login.c, src/suauth.c,
  src/faillog.c, src/groups.c, src/chsh.c, src/id.c: Review all the
  usage of one of the getpwnam(), getpwuid(), getgrnam(),
  getgrgid(), and getspnam() functions. It was noticed on
  http://bugs.debian.org/341230 that chfn and chsh use a passwd
  structure after calling a pam function, which result in using
  information from the passwd structure requested by pam, not the
  original one. It is much easier to use the new xget... functions
  to avoid these issues. I've checked which call to the original
  get... functions could be left (reducing the scope of the
  structure if possible), and I've left comments to ease future
  reviews (e.g. /* local, no need for xgetpwnam */).
  Note: the getpwent/getgrent calls should probably be checked also.
* src/groupdel.c, src/expiry.c: Fix typos in comments.
* src/groupmod.c: Re-indent.
* libmisc/Makefile.am, lib/groupmem.c, lib/groupio.c, lib/pwmem.c,
  lib/pwio.c, lib/shadowmem.c, lib/shadowio.c: Move the __<xx>_dup
  functions (used by the xget... functions) from the <xx>io.c files
  to the new <xx>mem.c files. This avoid linking some utils against
  the SELinux library.
49 files changed:
ChangeLog
configure.in
lib/groupio.c
lib/groupmem.c [new file with mode: 0644]
lib/prototypes.h
lib/pwio.c
lib/pwmem.c [new file with mode: 0644]
lib/shadowio.c
lib/shadowmem.c [new file with mode: 0644]
libmisc/Makefile.am
libmisc/addgrps.c
libmisc/chowntty.c
libmisc/entry.c
libmisc/limits.c
libmisc/myname.c
libmisc/pwdcheck.c
libmisc/rlogin.c
libmisc/xgetXXbyYY.c [new file with mode: 0644]
libmisc/xgetgrgid.c [new file with mode: 0644]
libmisc/xgetgrnam.c [new file with mode: 0644]
libmisc/xgetpwnam.c [new file with mode: 0644]
libmisc/xgetpwuid.c [new file with mode: 0644]
libmisc/xgetspnam.c [new file with mode: 0644]
src/chage.c
src/chfn.c
src/chgpasswd.c
src/chpasswd.c
src/chsh.c
src/expiry.c
src/faillog.c
src/gpasswd.c
src/groupadd.c
src/groupdel.c
src/groupmems.c
src/groupmod.c
src/groups.c
src/grpck.c
src/id.c
src/lastlog.c
src/login_nopam.c
src/newgrp.c
src/newusers.c
src/passwd.c
src/pwck.c
src/su.c
src/suauth.c
src/useradd.c
src/userdel.c
src/usermod.c