From 5e1c9b562c4bf89d0d2e9a423a6fa0bf48b6da6b Mon Sep 17 00:00:00 2001
From: Matthias Dieter Wallnöfer <mdw@samba.org>
Date: Fri, 3 Dec 2010 09:18:23 +0100
Subject: s4:auth/sam.c - fix the free of memory contexts

"tmp_ctx" needs always to be freed ("res" is freed implicitly)
---
 source4/auth/sam.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

(limited to 'source4')

diff --git a/source4/auth/sam.c b/source4/auth/sam.c
index cbaa2699c7..bd9278036e 100644
--- a/source4/auth/sam.c
+++ b/source4/auth/sam.c
@@ -353,6 +353,7 @@ NTSTATUS authsam_expand_nested_groups(struct ldb_context *sam_ctx,
 		already_there = sids_contains_sid((const struct dom_sid**) *res_sids,
 						  *num_res_sids, &sid);
 		if (already_there) {
+			talloc_free(tmp_ctx);
 			return NT_STATUS_OK;
 		}
 
@@ -396,7 +397,6 @@ NTSTATUS authsam_expand_nested_groups(struct ldb_context *sam_ctx,
 		status = authsam_expand_nested_groups(sam_ctx, &el->values[i],
 						      false, filter, res_sids_ctx, res_sids, num_res_sids);
 		if (!NT_STATUS_IS_OK(status)) {
-			talloc_free(res);
 			talloc_free(tmp_ctx);
 			return status;
 		}
-- 
cgit