diff options
| author | Stephen Gallagher <sgallagh@redhat.com> | 2015-04-14 13:07:36 -0400 |
|---|---|---|
| committer | Jakub Hrozek <jhrozek@redhat.com> | 2015-04-15 17:30:14 +0200 |
| commit | d9079aa05eb8aacb488992fdce328c1abadd08d8 (patch) | |
| tree | 573fba0373f4e2054d3720cbefedd1c7d16b187f | |
| parent | 1e0fa55fb377db788e065de917ba8e149eb56161 (diff) | |
| download | sssd-d9079aa05eb8aacb488992fdce328c1abadd08d8.tar.gz sssd-d9079aa05eb8aacb488992fdce328c1abadd08d8.tar.xz sssd-d9079aa05eb8aacb488992fdce328c1abadd08d8.zip | |
AD: Clean up ad_access_gpo
Align goto usage with conventions in the rest of the source.
Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
| -rw-r--r-- | src/providers/ad/ad_gpo.c | 12 |
1 files changed, 5 insertions, 7 deletions
diff --git a/src/providers/ad/ad_gpo.c b/src/providers/ad/ad_gpo.c index b71b4c2e9..971370803 100644 --- a/src/providers/ad/ad_gpo.c +++ b/src/providers/ad/ad_gpo.c @@ -1534,8 +1534,6 @@ ad_gpo_access_send(TALLOC_CTX *mem_ctx, if (gpo_map_type == GPO_MAP_PERMIT) { ret = EOK; - tevent_req_done(req); - tevent_req_post(req, ev); goto immediately; } @@ -1551,8 +1549,6 @@ ad_gpo_access_send(TALLOC_CTX *mem_ctx, "ad_gpo_access_control option were set to enforcing " \ "mode."); ret = EOK; - tevent_req_done(req); - tevent_req_post(req, ev); goto immediately; default: ret = EINVAL; @@ -1592,15 +1588,17 @@ ad_gpo_access_send(TALLOC_CTX *mem_ctx, } tevent_req_set_callback(subreq, ad_gpo_connect_done, req); - ret = EOK; + return req; immediately: - if (ret != EOK) { + if (ret == EOK) { + tevent_req_done(req); + } else { tevent_req_error(req, ret); - tevent_req_post(req, ev); } + tevent_req_post(req, ev); return req; } |