summaryrefslogtreecommitdiff
path: root/modules
diff options
context:
space:
mode:
authorRobin Hack <rhack@redhat.com>2014-08-25 17:33:21 +0200
committerTomas Mraz <tmraz@fedoraproject.org>2014-08-25 17:33:21 +0200
commit29b63ae08f071aee5b1b901e6fb8f8131b82032d (patch)
treea36badd0b1e624489c7966c7f4e358d63ef3f4b8 /modules
parentbbe948ae58314ce3395d39e0fe49cdba309c29df (diff)
pam_keyinit: Check return value of setregid.
modules/pam_keyinit/pam_keyinit.c (pam_sm_open_session): Log if setregid() fails.
Diffstat (limited to 'modules')
-rw-r--r--modules/pam_keyinit/pam_keyinit.c3
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;
}