diff options
author | Pavel Reichl <preichl@redhat.com> | 2015-02-25 05:38:38 -0500 |
---|---|---|
committer | Jakub Hrozek <jhrozek@redhat.com> | 2015-02-27 10:40:03 +0100 |
commit | 45b7cadbbd81df4a6aba39de97eb1522774bcc60 (patch) | |
tree | e2f0c6a4169728e8d1dea377d8c4efe890e3efda /src | |
parent | 09c9dfa2c3f183dfd2704e4dfd488521b793ae3a (diff) | |
download | sssd-45b7cadbbd81df4a6aba39de97eb1522774bcc60.tar.gz sssd-45b7cadbbd81df4a6aba39de97eb1522774bcc60.tar.xz sssd-45b7cadbbd81df4a6aba39de97eb1522774bcc60.zip |
PAM: check return value of confdb_get_string
Coverity found this neglect.
Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
(cherry picked from commit c5290f2175845f2c5e3f35ce279b6f52b1d51275)
Diffstat (limited to 'src')
-rw-r--r-- | src/responder/pam/pamsrv_cmd.c | 6 |
1 files changed, 6 insertions, 0 deletions
diff --git a/src/responder/pam/pamsrv_cmd.c b/src/responder/pam/pamsrv_cmd.c index ab8369b0f..86e763f82 100644 --- a/src/responder/pam/pamsrv_cmd.c +++ b/src/responder/pam/pamsrv_cmd.c @@ -679,6 +679,12 @@ static void pam_reply(struct pam_auth_req *preq) ret = confdb_get_string(pctx->rctx->cdb, pd, CONFDB_PAM_CONF_ENTRY, CONFDB_PAM_ACCOUNT_EXPIRED_MESSAGE, "", &pam_account_expired_message); + if (ret != EOK) { + DEBUG(SSSDBG_MINOR_FAILURE, + "Failed to get expiration message: %d:[%s].\n", + ret, sss_strerror(ret)); + goto done; + } inform_account_expired(pd, pam_account_expired_message); } |