From: Ondřej Surý <ondrej@isc.org>
Date: Wed, 23 Feb 2022 21:04:05 +0000 (+0100)
Subject: Properly free up enqueued netievents in nm_destroy()
X-Git-Tag: v9.19.0~100^2
X-Git-Url: http://git.ipfire.org/gitweb.cgi?a=commitdiff_plain;h=88418c33729804cc86a9492e3e30f1123f56ddcd;p=thirdparty%2Fbind9.git

Properly free up enqueued netievents in nm_destroy()

When the isc_netmgr is being destroyed, the normal and priority queues
should be dequeued and netievents properly freed.  This wasn't the case.
---

diff --git a/lib/isc/netmgr/netmgr.c b/lib/isc/netmgr/netmgr.c
index 4b4871de9c7..0e6c37fd7da 100644
--- a/lib/isc/netmgr/netmgr.c
+++ b/lib/isc/netmgr/netmgr.c
@@ -372,14 +372,14 @@ nm_destroy(isc_nm_t **mgr0) {
 
 		/* Empty the async event queues */
 		while ((ievent = DEQUEUE_PRIORITY_NETIEVENT(worker)) != NULL) {
-			isc_mem_put(mgr->mctx, ievent, sizeof(*ievent));
+			isc__nm_put_netievent(mgr, ievent);
 		}
 
 		INSIST(DEQUEUE_PRIVILEGED_NETIEVENT(worker) == NULL);
 		INSIST(DEQUEUE_TASK_NETIEVENT(worker) == NULL);
 
-		while ((ievent = DEQUEUE_PRIORITY_NETIEVENT(worker)) != NULL) {
-			isc_mem_put(mgr->mctx, ievent, sizeof(*ievent));
+		while ((ievent = DEQUEUE_NORMAL_NETIEVENT(worker)) != NULL) {
+			isc__nm_put_netievent(mgr, ievent);
 		}
 		isc_condition_destroy(&worker->cond_prio);
 		isc_mutex_destroy(&worker->lock);