>From f5cc688bc5095098e0e1585e58fd4a06f891a0d6 Mon Sep 17 00:00:00 2001 From: Sumit Bose Date: Mon, 23 Nov 2009 22:32:25 +0100 Subject: [PATCH] Really check return value from pam_set_item --- server/providers/proxy.c | 6 +++--- 1 files changed, 3 insertions(+), 3 deletions(-) diff --git a/server/providers/proxy.c b/server/providers/proxy.c index 9cf0c61..f55b1c2 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))); } -- 1.6.2.5