diff options
| author | Sumit Bose <sbose@redhat.com> | 2009-11-23 22:32:25 +0100 |
|---|---|---|
| committer | Stephen Gallagher <sgallagh@redhat.com> | 2009-11-23 16:49:51 -0500 |
| commit | f542305b2212795d82e530fdb9817abcb4002c93 (patch) | |
| tree | a78b3df68df24f60a8a81c8fd7965d352bdb4fab /server/providers | |
| parent | 089077a742b2cb6fbac3ab6c9c59b09dc6339247 (diff) | |
| download | sssd-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.c | 6 |
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))); } |