Attachment 13927 Details for Bug 13244 – Proposed git-am fix for master.

[patch] Proposed git-am fix for master.

0001-s3-ldap-Ensure-the-ADS_STRUCT-pointer-doesn-t-get-fr.patch (text/plain), 1.19 KB, created by Jeremy Allison on 2018-01-24 22:21:41 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Jeremy Allison

Created: 2018-01-24 22:21:41 UTC

Size: 1.19 KB

patch

obsolete

>From 7e607231456097437ceab8c77aa8f31e79c51350 Mon Sep 17 00:00:00 2001
>From: Jeremy Allison <jra@samba.org>
>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 <jra@samba.org>
>---
> 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
>

Actions: View

Attachments on bug 13244: 13927 | 13936