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_selinux.c | |
| parent | ccc2af010bbbe6d8a7496fb717216135bc4c1993 (diff) | |
| download | sssd-8e5549e453558d4bebdec333a93e215d5d6ffaec.tar.gz sssd-8e5549e453558d4bebdec333a93e215d5d6ffaec.tar.xz 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_selinux.c')
| -rw-r--r-- | src/providers/ipa/ipa_selinux.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/src/providers/ipa/ipa_selinux.c b/src/providers/ipa/ipa_selinux.c index e130c4c87..8ed17a2f6 100644 --- a/src/providers/ipa/ipa_selinux.c +++ b/src/providers/ipa/ipa_selinux.c @@ -120,7 +120,7 @@ void ipa_selinux_handler(struct be_req *be_req) return; fail: - be_req->fn(be_req, DP_ERR_FATAL, PAM_SYSTEM_ERR, NULL); + be_req_terminate(be_req, DP_ERR_FATAL, PAM_SYSTEM_ERR, NULL); } static struct ipa_selinux_op_ctx * @@ -248,7 +248,7 @@ static void ipa_selinux_handler_done(struct tevent_req *req) in_transaction = false; pd->pam_status = PAM_SUCCESS; - breq->fn(breq, DP_ERR_OK, EOK, "Success"); + be_req_terminate(breq, DP_ERR_OK, EOK, "Success"); return; fail: @@ -259,9 +259,9 @@ fail: } } if (ret == EAGAIN) { - breq->fn(breq, DP_ERR_OFFLINE, EAGAIN, "Offline"); + be_req_terminate(breq, DP_ERR_OFFLINE, EAGAIN, "Offline"); } else { - breq->fn(breq, DP_ERR_FATAL, ret, NULL); + be_req_terminate(breq, DP_ERR_FATAL, ret, NULL); } } |