From: Mark Andrews <marka@isc.org>
Date: Tue, 17 Jan 2023 06:25:25 +0000 (+1100)
Subject: isc_nm_listenudp: treat socket failures gracefully
X-Git-Tag: v9.19.10~25^2~3
X-Git-Url: http://git.ipfire.org/gitweb.cgi?a=commitdiff_plain;h=624f5a0daecc3f4d461c654ce8853edc975b6808;p=thirdparty%2Fbind9.git

isc_nm_listenudp: treat socket failures gracefully

The old code didn't handle race conditions and errors on systems
with non load balancing sockets gracefully.  Look for an error on
any child socket and if found close all the child sockets and return
an error.
---

diff --git a/lib/isc/netmgr/udp.c b/lib/isc/netmgr/udp.c
index 9072d688b46..75adf446715 100644
--- a/lib/isc/netmgr/udp.c
+++ b/lib/isc/netmgr/udp.c
@@ -177,8 +177,16 @@ isc_nm_listenudp(isc_nm_t *mgr, uint32_t workers, isc_sockaddr_t *iface,
 		isc__nm_closesocket(fd);
 	}
 
+	/*
+	 * If any of the child sockets have failed then isc_nm_listenudp
+	 * fails.
+	 */
 	for (size_t i = 1; i < sock->nchildren; i++) {
-		INSIST(result == sock->children[i].result);
+		if (result == ISC_R_SUCCESS &&
+		    sock->children[i].result != ISC_R_SUCCESS)
+		{
+			result = sock->children[i].result;
+		}
 	}
 
 	atomic_store(&sock->active, true);