diff options
| author | Jeremy Allison <jra@samba.org> | 2004-05-18 01:05:59 +0000 |
|---|---|---|
| committer | Gerald (Jerry) Carter <jerry@samba.org> | 2007-10-10 10:51:37 -0500 |
| commit | 05bc3279906d63f30df1c373a50c521a171225ab (patch) | |
| tree | 81f8591e5414da1528e2fdda2fc9b5d46af6782b | |
| parent | 49f95e6d27ac9476e8308e53629e8ae4908957c6 (diff) | |
| download | samba-05bc3279906d63f30df1c373a50c521a171225ab.tar.gz samba-05bc3279906d63f30df1c373a50c521a171225ab.tar.bz2 samba-05bc3279906d63f30df1c373a50c521a171225ab.zip | |
r764: More memleak fixes in error code path from kawasa_r@itg.hitachi.co.jp.
Jeremy.
(This used to be commit 9647394e7c79c81ac4cf276a2c4b9e16eb053ec2)
| -rw-r--r-- | source3/libads/ldap.c | 7 |
1 files changed, 6 insertions, 1 deletions
diff --git a/source3/libads/ldap.c b/source3/libads/ldap.c index e156857e96..e018eeb2da 100644 --- a/source3/libads/ldap.c +++ b/source3/libads/ldap.c @@ -1980,17 +1980,22 @@ ADS_STATUS ads_server_info(ADS_STRUCT *ads) } status = ads_do_search(ads, "", LDAP_SCOPE_BASE, "(objectclass=*)", attrs, &res); - if (!ADS_ERR_OK(status)) return status; + if (!ADS_ERR_OK(status)) { + talloc_destroy(ctx); + return status; + } value = ads_pull_string(ads, ctx, res, "ldapServiceName"); if (!value) { ads_msgfree(ads, res); + talloc_destroy(ctx); return ADS_ERROR(LDAP_NO_RESULTS_RETURNED); } timestr = ads_pull_string(ads, ctx, res, "currentTime"); if (!timestr) { ads_msgfree(ads, res); + talloc_destroy(ctx); return ADS_ERROR(LDAP_NO_RESULTS_RETURNED); } |