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:43:06 +0200 |
| commit | 9fadb9c95da9175c4a312690d7d06908c1f7e424 (patch) | |
| tree | 3e00b81fefca7ee1cd12243a4423b8365f3eeff9 /src | |
| parent | 8ae3dfccf2d77f69135d45004626792128883d2f (diff) | |
| download | sssd-9fadb9c95da9175c4a312690d7d06908c1f7e424.tar.gz sssd-9fadb9c95da9175c4a312690d7d06908c1f7e424.tar.xz sssd-9fadb9c95da9175c4a312690d7d06908c1f7e424.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>
(cherry picked from commit d9079aa05eb8aacb488992fdce328c1abadd08d8)
(cherry picked from commit d7efa39ab732fb034f51501cb2b1b8d3b1716979)
Diffstat (limited to 'src')
| -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 69ff6daaa..bd1244ae4 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; } |