summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorAndrew Bartlett <abartlet@samba.org>2004-01-02 11:39:07 +0000
committerAndrew Bartlett <abartlet@samba.org>2004-01-02 11:39:07 +0000
commitfd346ce0c9b2deccc8164bec99dbd60cff8101be (patch)
treeacce65a5d091eccfa43e938d74cf82b7d0148f14
parentffedf5a1aa4d1c08a6d2ba92afbb933e21f6cf55 (diff)
downloadsamba-fd346ce0c9b2deccc8164bec99dbd60cff8101be.tar.gz
samba-fd346ce0c9b2deccc8164bec99dbd60cff8101be.tar.bz2
samba-fd346ce0c9b2deccc8164bec99dbd60cff8101be.zip
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)
-rw-r--r--source3/nsswitch/winbindd_ads.c2
1 files changed, 0 insertions, 2 deletions
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;
}