diff options
author | Simo Sorce <simo@redhat.com> | 2013-01-11 12:25:53 -0500 |
---|---|---|
committer | Jakub Hrozek <jhrozek@redhat.com> | 2013-01-21 22:17:34 +0100 |
commit | 8e5549e453558d4bebdec333a93e215d5d6ffaec (patch) | |
tree | a9004b07b800492ecb8bdaeb2019512c3fccb229 /src/providers/ipa/ipa_access.c | |
parent | ccc2af010bbbe6d8a7496fb717216135bc4c1993 (diff) | |
download | sssd-8e5549e453558d4bebdec333a93e215d5d6ffaec.tar.gz sssd-8e5549e453558d4bebdec333a93e215d5d6ffaec.tar.bz2 sssd-8e5549e453558d4bebdec333a93e215d5d6ffaec.zip |
Introduce be_req_terminate() helper
Call it everywhere instead of directly dereferencing be_req->fn
This is in preparation of making be_req opaque.
Diffstat (limited to 'src/providers/ipa/ipa_access.c')
-rw-r--r-- | src/providers/ipa/ipa_access.c | 12 |
1 files changed, 6 insertions, 6 deletions
diff --git a/src/providers/ipa/ipa_access.c b/src/providers/ipa/ipa_access.c index 927a9cbc..1b626029 100644 --- a/src/providers/ipa/ipa_access.c +++ b/src/providers/ipa/ipa_access.c @@ -46,9 +46,9 @@ static void ipa_access_reply(struct hbac_ctx *hbac_ctx, int pam_status) talloc_zfree(hbac_ctx); if (pam_status == PAM_SUCCESS || pam_status == PAM_PERM_DENIED) { - be_req->fn(be_req, DP_ERR_OK, pam_status, NULL); + be_req_terminate(be_req, DP_ERR_OK, pam_status, NULL); } else { - be_req->fn(be_req, DP_ERR_FATAL, pam_status, NULL); + be_req_terminate(be_req, DP_ERR_FATAL, pam_status, NULL); } } @@ -96,7 +96,7 @@ void ipa_access_handler(struct be_req *be_req) ipa_access_ctx->sdap_access_ctx, pd); if (!req) { - be_req->fn(be_req, DP_ERR_FATAL, PAM_SYSTEM_ERR, NULL); + be_req_terminate(be_req, DP_ERR_FATAL, PAM_SYSTEM_ERR, NULL); return; } tevent_req_set_callback(req, ipa_hbac_check, be_req); @@ -130,12 +130,12 @@ static void ipa_hbac_check(struct tevent_req *req) * here. */ pd->pam_status = PAM_PERM_DENIED; - be_req->fn(be_req, DP_ERR_OK, PAM_PERM_DENIED, NULL); + be_req_terminate(be_req, DP_ERR_OK, PAM_PERM_DENIED, NULL); return; default: /* We got an unexpected error. Return it as-is */ pd->pam_status = PAM_SYSTEM_ERR; - be_req->fn(be_req, DP_ERR_FATAL, pam_status, NULL); + be_req_terminate(be_req, DP_ERR_FATAL, pam_status, NULL); return; } @@ -180,7 +180,7 @@ fail: /* Return an proper error */ ipa_access_reply(hbac_ctx, pam_status); } else { - be_req->fn(be_req, DP_ERR_FATAL, pam_status, NULL); + be_req_terminate(be_req, DP_ERR_FATAL, pam_status, NULL); } } |