Hello,

this patch fixes annoying bug from ticket #55: BIND cannot be shutdown correctly, if psearch is enabled and LDAP connect fails

Upstream ticket: https://fedorahosted.org/bind-dyndb-ldap/ticket/55

It's trivial three line patch, so Adam ACKed it on IRC.

Patch is already in master: https://fedorahosted.org/bind-dyndb-ldap/changeset/55d2886bb016adf3081a25eea616ce40f17d03e4

I think it's annoying and fix is trivial, so I opened BZ ticket for RHEL6.3: https://bugzilla.redhat.com/show_bug.cgi?id=802375

--
Petr^2 Spacek
>From 55d2886bb016adf3081a25eea616ce40f17d03e4 Mon Sep 17 00:00:00 2001
From: Petr Spacek <pspa...@redhat.com>
Date: Mon, 12 Mar 2012 13:01:25 +0100
Subject: [PATCH] Fix ticket #55: BIND cannot be shutdown correctly, if
 psearch is enabled and LDAP connect fails Signed-off-by:
 Petr Spacek <pspa...@redhat.com>

---
 src/ldap_helper.c |    3 +++
 1 files changed, 3 insertions(+), 0 deletions(-)

diff --git a/src/ldap_helper.c b/src/ldap_helper.c
index eff7b3d..0f01ec2 100644
--- a/src/ldap_helper.c
+++ b/src/ldap_helper.c
@@ -3063,6 +3063,9 @@ ldap_psearch_watcher(isc_threadarg_t arg)
 
 	/* Try to connect. */
 	while (conn->handle == NULL) {
+		if (inst->exiting)
+				goto cleanup;
+
 		log_error("ldap_psearch_watcher handle is NULL. "
 		          "Next try in %ds", inst->reconnect_interval);
 		sleep(inst->reconnect_interval);
-- 
1.7.7.6

_______________________________________________
Freeipa-devel mailing list
Freeipa-devel@redhat.com
https://www.redhat.com/mailman/listinfo/freeipa-devel

Reply via email to