diff options
| author | Jakub Hrozek <jhrozek@redhat.com> | 2011-02-15 14:16:13 +0100 |
|---|---|---|
| committer | Stephen Gallagher <sgallagh@redhat.com> | 2011-04-08 11:32:23 -0400 |
| commit | df33bc1ede96f57f2b8be1fa6f8a1f7451bf051f (patch) | |
| tree | 6a24201d0c075dd13459ec8f888336edc5c5ab2c /src/providers/ldap/ldap_auth.c | |
| parent | 4a28fb10122bd74ba33607af46f028813de9161d (diff) | |
| download | sssd-df33bc1ede96f57f2b8be1fa6f8a1f7451bf051f.tar.gz sssd-df33bc1ede96f57f2b8be1fa6f8a1f7451bf051f.tar.xz sssd-df33bc1ede96f57f2b8be1fa6f8a1f7451bf051f.zip | |
Fix unchecked return values of pam_add_response
https://fedorahosted.org/sssd/ticket/798
Diffstat (limited to 'src/providers/ldap/ldap_auth.c')
| -rw-r--r-- | src/providers/ldap/ldap_auth.c | 9 |
1 files changed, 7 insertions, 2 deletions
diff --git a/src/providers/ldap/ldap_auth.c b/src/providers/ldap/ldap_auth.c index e45d5b3ed..fd43c4321 100644 --- a/src/providers/ldap/ldap_auth.c +++ b/src/providers/ldap/ldap_auth.c @@ -206,6 +206,7 @@ static errno_t check_pwexpire_ldap(struct pam_data *pd, if (ppolicy->grace > 0 || ppolicy->expire > 0) { uint32_t *data; uint32_t *ptr; + int ret; data = talloc_size(pd, 2* sizeof(uint32_t)); if (data == NULL) { @@ -224,8 +225,12 @@ static errno_t check_pwexpire_ldap(struct pam_data *pd, *ptr = ppolicy->expire; } - pam_add_response(pd, SSS_PAM_USER_INFO, 2* sizeof(uint32_t), - (uint8_t*)data); + ret = pam_add_response(pd, SSS_PAM_USER_INFO, 2* sizeof(uint32_t), + (uint8_t*)data); + if (ret != EOK) { + DEBUG(1, ("pam_add_response failed.\n")); + return ret; + } } *result = SDAP_AUTH_SUCCESS; |