diff options
| author | Robin Hack <rhack@redhat.com> | 2014-08-25 17:33:21 +0200 |
|---|---|---|
| committer | Tomas Mraz <tmraz@fedoraproject.org> | 2014-08-25 17:33:21 +0200 |
| commit | 29b63ae08f071aee5b1b901e6fb8f8131b82032d (patch) | |
| tree | a36badd0b1e624489c7966c7f4e358d63ef3f4b8 | |
| parent | bbe948ae58314ce3395d39e0fe49cdba309c29df (diff) | |
| download | pam-29b63ae08f071aee5b1b901e6fb8f8131b82032d.tar.gz pam-29b63ae08f071aee5b1b901e6fb8f8131b82032d.tar.bz2 pam-29b63ae08f071aee5b1b901e6fb8f8131b82032d.zip | |
pam_keyinit: Check return value of setregid.
modules/pam_keyinit/pam_keyinit.c (pam_sm_open_session): Log if setregid() fails.
| -rw-r--r-- | modules/pam_keyinit/pam_keyinit.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/modules/pam_keyinit/pam_keyinit.c b/modules/pam_keyinit/pam_keyinit.c index 8d0501e0..f82eead2 100644 --- a/modules/pam_keyinit/pam_keyinit.c +++ b/modules/pam_keyinit/pam_keyinit.c @@ -218,7 +218,8 @@ int pam_sm_open_session(pam_handle_t *pamh, int flags UNUSED, if (uid != old_uid && setreuid(uid, -1) < 0) { error(pamh, "Unable to change UID to %d temporarily\n", uid); - setregid(old_gid, -1); + if (setregid(old_gid, -1) < 0) + error(pamh, "Unable to change GID back to %d\n", old_gid); return PAM_SESSION_ERR; } |