From 7e607231456097437ceab8c77aa8f31e79c51350 Mon Sep 17 00:00:00 2001 From: Jeremy Allison Date: Wed, 24 Jan 2018 14:09:43 -0800 Subject: [PATCH] s3: ldap: Ensure the ADS_STRUCT pointer doesn't get freed on error, we don't own it here. BUG: https://bugzilla.samba.org/show_bug.cgi?id=13244 Signed-off-by: Jeremy Allison --- source3/libads/ldap_utils.c | 9 +++++++++ 1 file changed, 9 insertions(+) diff --git a/source3/libads/ldap_utils.c b/source3/libads/ldap_utils.c index a4adbc09524..0c37b06500c 100644 --- a/source3/libads/ldap_utils.c +++ b/source3/libads/ldap_utils.c @@ -105,9 +105,18 @@ static ADS_STATUS ads_do_search_retry_internal(ADS_STRUCT *ads, const char *bind status = ads_connect(ads); if (!ADS_ERR_OK(status)) { + bool orig_is_mine = ads->is_mine; + DEBUG(1,("ads_search_retry: failed to reconnect (%s)\n", ads_errstr(status))); + /* + * We need to keep the ads pointer + * from being freed here as we don't own it and + * callers depend on it being around. + */ + ads->is_mine = false; ads_destroy(&ads); + ads->is_mine = orig_is_mine; SAFE_FREE(bp); return status; } -- 2.16.0.rc1.238.g530d649a79-goog