From f542305b2212795d82e530fdb9817abcb4002c93 Mon Sep 17 00:00:00 2001 From: Sumit Bose Date: Mon, 23 Nov 2009 22:32:25 +0100 Subject: Really check return value from pam_set_item --- server/providers/proxy.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) (limited to 'server') diff --git a/server/providers/proxy.c b/server/providers/proxy.c index 9cf0c616a..f55b1c227 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))); } -- cgit