From: Andreas Gustafsson <source@isc.org>
Date: Wed, 29 Aug 2001 23:52:28 +0000 (+0000)
Subject: The request object would fail to send its completion event
X-Git-Tag: v9.2.0b1~1^2~4
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=8068ceb2a9cc56d18016c3cd94a09e4bb0bc7b0d;p=thirdparty%2Fbind9.git

The request object would fail to send its completion event
when the control event was delivered if a response was received between sending and delivery
of the completion event [RT #1676]
---

diff --git a/lib/dns/request.c b/lib/dns/request.c
index 44a2fb6a9f9..4a24950848d 100644
--- a/lib/dns/request.c
+++ b/lib/dns/request.c
@@ -15,7 +15,7 @@
  * WITH THE USE OR PERFORMANCE OF THIS SOFTWARE.
  */
 
-/* $Id: request.c,v 1.65 2001/08/29 21:15:56 gson Exp $ */
+/* $Id: request.c,v 1.66 2001/08/29 23:52:28 gson Exp $ */
 
 #include <config.h>
 
@@ -89,7 +89,8 @@ struct dns_request {
 
 #define DNS_REQUEST_F_CONNECTING 0x0001
 #define DNS_REQUEST_F_SENDING 0x0002
-#define DNS_REQUEST_F_CANCELED 0x0004
+#define DNS_REQUEST_F_CANCELED 0x0004	/* ctlevent received, or otherwise
+					   synchronously canceled */
 #define DNS_REQUEST_F_TIMEDOUT 0x0008	/* cancelled due to a timeout */
 #define DNS_REQUEST_F_TCP 0x0010	/* This request used TCP */
 #define DNS_REQUEST_CANCELED(r) \
@@ -251,8 +252,6 @@ static void
 mgr_shutdown(dns_requestmgr_t *requestmgr) {
 	dns_request_t *request;
 
-	req_log(ISC_LOG_DEBUG(3), "mgr_shutdown: %p", requestmgr);
-
 	/*
 	 * Caller holds lock.
 	 */
@@ -1042,10 +1041,9 @@ do_cancel(isc_task_t *task, isc_event_t *event) {
 	INSIST(event->ev_type == DNS_EVENT_REQUESTCONTROL);
 	LOCK(&request->requestmgr->locks[request->hash]);
 	request->canceling = ISC_FALSE;
-	if (!DNS_REQUEST_CANCELED(request)) {
+	if (!DNS_REQUEST_CANCELED(request))
 		req_cancel(request);
-		send_if_done(request, ISC_R_CANCELED);
-	}
+	send_if_done(request, ISC_R_CANCELED);
 	UNLOCK(&request->requestmgr->locks[request->hash]);	
 }
 
@@ -1313,6 +1311,9 @@ req_destroy(dns_request_t *request) {
 	isc_mem_detach(&mctx);
 }
 
+/*
+ * Stop the current request.  Must be called from the request's task.
+ */
 static void
 req_cancel(dns_request_t *request) {
 	isc_socket_t *socket;