diff options
author | Andrew Bartlett <abartlet@samba.org> | 2004-01-02 11:39:07 +0000 |
---|---|---|
committer | Andrew Bartlett <abartlet@samba.org> | 2004-01-02 11:39:07 +0000 |
commit | fd346ce0c9b2deccc8164bec99dbd60cff8101be (patch) | |
tree | acce65a5d091eccfa43e938d74cf82b7d0148f14 | |
parent | ffedf5a1aa4d1c08a6d2ba92afbb933e21f6cf55 (diff) | |
download | samba-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.c | 2 |
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; } |