diff options
| author | Simo Sorce <simo@redhat.com> | 2013-03-03 18:06:13 -0500 |
|---|---|---|
| committer | Simo Sorce <simo@redhat.com> | 2013-03-13 14:39:50 -0400 |
| commit | 12150d43f17feff531ce6c59c91653205f889649 (patch) | |
| tree | 2859cc3e7accedad7365ff822b0769ea5c7285ba /src/providers/ldap/ldap_access.c | |
| parent | d5d0fb8278afa5bb95bffd8e4e9c8ac13c18419c (diff) | |
| download | sssd-err_facility.tar.gz sssd-err_facility.tar.xz sssd-err_facility.zip | |
Convert sdap_access to new error codeserr_facility
Also simplify sdap_access_send to avoid completely fake _send() routines.
Diffstat (limited to 'src/providers/ldap/ldap_access.c')
| -rw-r--r-- | src/providers/ldap/ldap_access.c | 14 |
1 files changed, 11 insertions, 3 deletions
diff --git a/src/providers/ldap/ldap_access.c b/src/providers/ldap/ldap_access.c index 5a8e12f0..4a06e66b 100644 --- a/src/providers/ldap/ldap_access.c +++ b/src/providers/ldap/ldap_access.c @@ -69,15 +69,23 @@ void sdap_pam_access_handler(struct be_req *breq) static void sdap_access_done(struct tevent_req *req) { errno_t ret; - int pam_status = PAM_SYSTEM_ERR; + int pam_status; struct be_req *breq = tevent_req_callback_data(req, struct be_req); - ret = sdap_access_recv(req, &pam_status); + ret = sdap_access_recv(req); talloc_zfree(req); - if (ret != EOK) { + switch (ret) { + case EOK: + pam_status = PAM_SUCCESS; + break; + case ERR_ACCESS_DENIED: + pam_status = PAM_PERM_DENIED; + break; + default: DEBUG(SSSDBG_CRIT_FAILURE, ("Error retrieving access check result.\n")); pam_status = PAM_SYSTEM_ERR; + break; } sdap_access_reply(breq, pam_status); |