From fd346ce0c9b2deccc8164bec99dbd60cff8101be Mon Sep 17 00:00:00 2001
From: Andrew Bartlett <abartlet@samba.org>
Date: Fri, 2 Jan 2004 11:39:07 +0000
Subject: Under certain error conditions (a talloc() failure above) this would
 cause a double-free(), and the resultant malloc heap corruption.

This may be one of our lurking winbind segfaults.

Andrew Bartlett
(This used to be commit 903263a1bdb755f86dac3a9a92a4af39c8b102c4)
---
 source3/nsswitch/winbindd_ads.c | 2 --
 1 file changed, 2 deletions(-)

(limited to 'source3')

diff --git a/source3/nsswitch/winbindd_ads.c b/source3/nsswitch/winbindd_ads.c
index 2fcf02a316..4b12304693 100644
--- a/source3/nsswitch/winbindd_ads.c
+++ b/source3/nsswitch/winbindd_ads.c
@@ -554,8 +554,6 @@ static NTSTATUS lookup_usergroups_alt(struct winbindd_domain *domain,
 done:
 	if (res) 
 		ads_msgfree(ads, res);
-	if (msg) 
-		ads_msgfree(ads, msg);
 
 	return status;
 }
-- 
cgit