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/data_provider_be.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/data_provider_be.c')
| -rw-r--r-- | src/providers/data_provider_be.c | 11 |
1 files changed, 9 insertions, 2 deletions
diff --git a/src/providers/data_provider_be.c b/src/providers/data_provider_be.c index 133959a18..818927c22 100644 --- a/src/providers/data_provider_be.c +++ b/src/providers/data_provider_be.c @@ -119,6 +119,13 @@ static struct bet_data bet_data[] = { {BET_MAX, NULL, NULL} }; +void be_req_terminate(struct be_req *be_req, + int dp_err_type, int errnum, const char *errstr) +{ + be_req->fn(be_req, dp_err_type, errnum, errstr); +} + + struct be_async_req { be_req_fn_t fn; struct be_req *req; @@ -2062,7 +2069,7 @@ static void be_target_access_permit(struct be_req *be_req) DEBUG(9, ("be_target_access_permit called, returning PAM_SUCCESS.\n")); pd->pam_status = PAM_SUCCESS; - be_req->fn(be_req, DP_ERR_OK, PAM_SUCCESS, NULL); + be_req_terminate(be_req, DP_ERR_OK, PAM_SUCCESS, NULL); } static struct bet_ops be_target_access_permit_ops = { @@ -2077,7 +2084,7 @@ static void be_target_access_deny(struct be_req *be_req) DEBUG(9, ("be_target_access_deny called, returning PAM_PERM_DENIED.\n")); 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); } static struct bet_ops be_target_access_deny_ops = { |