diff options
author | Pavel Reichl <preichl@redhat.com> | 2015-09-03 04:46:50 -0400 |
---|---|---|
committer | Lukas Slebodnik <lslebodn@redhat.com> | 2015-10-07 13:01:35 +0200 |
commit | 4a89c2ebbe2a98e790536165486ecf2d14836e69 (patch) | |
tree | a38effe6d7a099f18588f29f29dc733d29be1b0f | |
parent | a73c89f8672a12878a8668bc321e6742bc45b924 (diff) | |
download | sssd-4a89c2ebbe2a98e790536165486ecf2d14836e69.tar.gz sssd-4a89c2ebbe2a98e790536165486ecf2d14836e69.tar.xz sssd-4a89c2ebbe2a98e790536165486ecf2d14836e69.zip |
GPO: fix memory leaksssd-1-12
Resolves:
https://fedorahosted.org/sssd/ticket/2777
Reviewed-by: Michal Židek <mzidek@redhat.com>
(cherry picked from commit 5dbdcc2c7210a0e3eb60ad1e85ba33f27d7faeda)
-rw-r--r-- | src/providers/ad/ad_gpo.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/src/providers/ad/ad_gpo.c b/src/providers/ad/ad_gpo.c index af864dfbe..bde810a38 100644 --- a/src/providers/ad/ad_gpo.c +++ b/src/providers/ad/ad_gpo.c @@ -557,14 +557,14 @@ ad_gpo_get_sids(TALLOC_CTX *mem_ctx, DEBUG(SSSDBG_OP_FAILURE, "sysdb_initgroups failed: [%d](%s)\n", ret, sss_strerror(ret)); - return ret; + goto done; } if (res->count == 0) { ret = ENOENT; DEBUG(SSSDBG_OP_FAILURE, "sysdb_initgroups returned empty result\n"); - return ret; + goto done; } user_sid = ldb_msg_find_attr_as_string(res->msgs[0], SYSDB_SID_STR, NULL); @@ -599,7 +599,7 @@ ad_gpo_get_sids(TALLOC_CTX *mem_ctx, *_group_size = num_group_sids + 1; *_group_sids = talloc_steal(mem_ctx, group_sids); *_user_sid = talloc_steal(mem_ctx, user_sid); - return EOK; + ret = EOK; done: talloc_free(tmp_ctx); |