summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorPavel Zuna <pzuna@redhat.com>2011-10-05 12:54:46 -0400
committerStephen Gallagher <sgallagh@redhat.com>2011-10-06 10:57:02 -0400
commit73851471133f0345945a4232c5dc147be26f9941 (patch)
tree814de480882b029f2fd148243af68a7b8a1ed286
parentcc747b056d161cb4a82d94e95b7522ca98c5d085 (diff)
downloadsssd-73851471133f0345945a4232c5dc147be26f9941.tar.gz
sssd-73851471133f0345945a4232c5dc147be26f9941.tar.xz
sssd-73851471133f0345945a4232c5dc147be26f9941.zip
Fix small bug where TALLOC_CTX could end up unfreed.
-rw-r--r--src/providers/ldap/sdap_async_groups.c6
1 files changed, 3 insertions, 3 deletions
diff --git a/src/providers/ldap/sdap_async_groups.c b/src/providers/ldap/sdap_async_groups.c
index 2b48af928..1e8e4b3d8 100644
--- a/src/providers/ldap/sdap_async_groups.c
+++ b/src/providers/ldap/sdap_async_groups.c
@@ -1530,14 +1530,14 @@ static errno_t sdap_nested_group_populate_users(struct sysdb_ctx *sysdb,
static const char *search_attrs[] = { SYSDB_NAME, NULL };
size_t count;
- tmp_ctx = talloc_new(NULL);
- if (!tmp_ctx) return ENOMEM;
-
if (num_users == 0) {
/* Nothing to do if there are no users */
return EOK;
}
+ tmp_ctx = talloc_new(NULL);
+ if (!tmp_ctx) return ENOMEM;
+
ret = sysdb_transaction_start(sysdb);
if (ret) {
DEBUG(1, ("Failed to start transaction!\n"));