summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorPavel Reichl <preichl@redhat.com>2015-09-04 07:04:10 -0400
committerJakub Hrozek <jhrozek@redhat.com>2015-10-02 12:50:37 +0200
commit3fa03d5816d6a401d8e894b77236d3cfd95dbd96 (patch)
tree41e29ed9652de51acfbaf0e472504dab981e062a
parenta2d6d4db64a7c3b27dea22fe52245925d688bd2d (diff)
downloadsssd-3fa03d5816d6a401d8e894b77236d3cfd95dbd96.tar.gz
sssd-3fa03d5816d6a401d8e894b77236d3cfd95dbd96.tar.xz
sssd-3fa03d5816d6a401d8e894b77236d3cfd95dbd96.zip
SDAP: fix minor memory leak
Reviewed-by: Pavel Březina <pbrezina@redhat.com>
-rw-r--r--src/providers/ldap/sdap_async_groups.c2
-rw-r--r--src/providers/ldap/sdap_idmap.c3
2 files changed, 3 insertions, 2 deletions
diff --git a/src/providers/ldap/sdap_async_groups.c b/src/providers/ldap/sdap_async_groups.c
index 653187b3a..609668339 100644
--- a/src/providers/ldap/sdap_async_groups.c
+++ b/src/providers/ldap/sdap_async_groups.c
@@ -592,7 +592,7 @@ static int sdap_save_group(TALLOC_CTX *memctx,
if (ret != EOK) {
DEBUG(SSSDBG_OP_FAILURE,
"Error: Failed to mark group as non-posix!\n");
- return ret;
+ goto done;
}
}
diff --git a/src/providers/ldap/sdap_idmap.c b/src/providers/ldap/sdap_idmap.c
index dd959b2c1..36d529836 100644
--- a/src/providers/ldap/sdap_idmap.c
+++ b/src/providers/ldap/sdap_idmap.c
@@ -206,7 +206,8 @@ sdap_idmap_init(TALLOC_CTX *mem_ctx,
if (err != IDMAP_SUCCESS) {
/* This should never happen */
DEBUG(SSSDBG_CRIT_FAILURE, "sss_idmap_ctx corrupted\n");
- return EIO;
+ ret = EIO;
+ goto done;
}