diff options
| author | Jeremy Allison <jra@samba.org> | 2004-05-07 17:58:06 +0000 |
|---|---|---|
| committer | Gerald (Jerry) Carter <jerry@samba.org> | 2007-10-10 10:51:29 -0500 |
| commit | a442c65e59cd0b7667087f90522883475dfb7eee (patch) | |
| tree | 139c568a233ed7352df0a4c22f36ca158481960b /source3 | |
| parent | 88adbfa1f7e136705fad2fe2926e4053808914ab (diff) | |
| download | samba-a442c65e59cd0b7667087f90522883475dfb7eee.tar.gz samba-a442c65e59cd0b7667087f90522883475dfb7eee.tar.xz samba-a442c65e59cd0b7667087f90522883475dfb7eee.zip | |
r562: Memory leak fix in error code path from kawasa_r@itg.hitachi.co.jp.
Jeremy.
(This used to be commit ac501348f473045a7846ffd9bc6b9eb4682b8987)
Diffstat (limited to 'source3')
| -rw-r--r-- | source3/libads/ldap.c | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/source3/libads/ldap.c b/source3/libads/ldap.c index 20a36dfdf5c..e156857e96f 100644 --- a/source3/libads/ldap.c +++ b/source3/libads/ldap.c @@ -1984,15 +1984,17 @@ ADS_STATUS ads_server_info(ADS_STRUCT *ads) value = ads_pull_string(ads, ctx, res, "ldapServiceName"); if (!value) { + ads_msgfree(ads, res); return ADS_ERROR(LDAP_NO_RESULTS_RETURNED); } timestr = ads_pull_string(ads, ctx, res, "currentTime"); if (!timestr) { + ads_msgfree(ads, res); return ADS_ERROR(LDAP_NO_RESULTS_RETURNED); } - ldap_msgfree(res); + ads_msgfree(ads, res); p = strchr(value, ':'); if (!p) { @@ -2054,6 +2056,7 @@ ADS_STATUS ads_domain_sid(ADS_STRUCT *ads, DOM_SID *sid) attrs, &res); if (!ADS_ERR_OK(rc)) return rc; if (!ads_pull_sid(ads, res, "objectSid", sid)) { + ads_msgfree(ads, res); return ADS_ERROR_SYSTEM(ENOENT); } ads_msgfree(ads, res); |