summaryrefslogtreecommitdiff
path: root/server/providers
diff options
context:
space:
mode:
authorSumit Bose <sbose@redhat.com>2009-11-23 22:32:25 +0100
committerStephen Gallagher <sgallagh@redhat.com>2009-11-23 16:49:51 -0500
commitf542305b2212795d82e530fdb9817abcb4002c93 (patch)
treea78b3df68df24f60a8a81c8fd7965d352bdb4fab /server/providers
parent089077a742b2cb6fbac3ab6c9c59b09dc6339247 (diff)
downloadsssd-f542305b2212795d82e530fdb9817abcb4002c93.tar.gz
sssd-f542305b2212795d82e530fdb9817abcb4002c93.tar.bz2
sssd-f542305b2212795d82e530fdb9817abcb4002c93.zip
Really check return value from pam_set_item
Diffstat (limited to 'server/providers')
-rw-r--r--server/providers/proxy.c6
1 files changed, 3 insertions, 3 deletions
diff --git a/server/providers/proxy.c b/server/providers/proxy.c
index 9cf0c616..f55b1c22 100644
--- a/server/providers/proxy.c
+++ b/server/providers/proxy.c
@@ -158,15 +158,15 @@ static void proxy_pam_handler(struct be_req *req) {
ret = pam_start(ctx->pam_target, pd->user, &conv, &pamh);
if (ret == PAM_SUCCESS) {
DEBUG(1, ("Pam transaction started.\n"));
- pam_set_item(pamh, PAM_TTY, pd->tty);
+ ret = pam_set_item(pamh, PAM_TTY, pd->tty);
if (ret != PAM_SUCCESS) {
DEBUG(1, ("Setting PAM_TTY failed: %s.\n", pam_strerror(pamh, ret)));
}
- pam_set_item(pamh, PAM_RUSER, pd->ruser);
+ ret = pam_set_item(pamh, PAM_RUSER, pd->ruser);
if (ret != PAM_SUCCESS) {
DEBUG(1, ("Setting PAM_RUSER failed: %s.\n", pam_strerror(pamh, ret)));
}
- pam_set_item(pamh, PAM_RHOST, pd->rhost);
+ ret = pam_set_item(pamh, PAM_RHOST, pd->rhost);
if (ret != PAM_SUCCESS) {
DEBUG(1, ("Setting PAM_RHOST failed: %s.\n", pam_strerror(pamh, ret)));
}