Fix CVE-2023-3341

.bind.metadata

@@ -0,0 +1,2 @@
+30cbd1f3e9d2d47d653498143334128aac1f8fc0 bind-9.16.23.tar.xz
+d8bd9c9c851f7f3e9419c4ccbda25813a97893a9 bind-9.16.23.tar.xz.asc

bind-9.16-CVE-2023-3341.patch

@@ -0,0 +1,168 @@
+From b137e12dc8118cddee20e372e480a495585e72b6 Mon Sep 17 00:00:00 2001
+From: Petr Mensik <pemensik@redhat.com>
+Date: Tue, 19 Sep 2023 12:44:31 +0200
+Subject: [PATCH] Fix CVE-2023-3341
+
+6245.	[security]	Limit the amount of recursion that can be performed
+			by isccc_cc_fromwire. (CVE-2023-3341) [GL #4152]
+---
+ lib/isccc/cc.c                   | 39 ++++++++++++++++++++++++--------
+ lib/isccc/include/isccc/result.h |  4 +++-
+ lib/isccc/result.c               |  4 +++-
+ 3 files changed, 35 insertions(+), 12 deletions(-)
+
+diff --git a/lib/isccc/cc.c b/lib/isccc/cc.c
+index 0be28b9057..3744d0f037 100644
+--- a/lib/isccc/cc.c
++++ b/lib/isccc/cc.c
+@@ -50,6 +50,10 @@
+ 
+ #define MAX_TAGS     256
+ #define DUP_LIFETIME 900
++#ifndef ISCCC_MAXDEPTH
++#define ISCCC_MAXDEPTH \
++	10 /* Big enough for rndc which just sends a string each way. */
++#endif
+ 
+ typedef isccc_sexpr_t *sexpr_ptr;
+ 
+@@ -480,19 +484,25 @@ verify(isccc_sexpr_t *alist, unsigned char *data, unsigned int length,
+ 
+ static isc_result_t
+ table_fromwire(isccc_region_t *source, isccc_region_t *secret,
+-	       uint32_t algorithm, isccc_sexpr_t **alistp);
++	       uint32_t algorithm, unsigned int depth, isccc_sexpr_t **alistp);
+ 
+ static isc_result_t
+-list_fromwire(isccc_region_t *source, isccc_sexpr_t **listp);
++list_fromwire(isccc_region_t *source, unsigned int depth,
++	      isccc_sexpr_t **listp);
+ 
+ static isc_result_t
+-value_fromwire(isccc_region_t *source, isccc_sexpr_t **valuep) {
++value_fromwire(isccc_region_t *source, unsigned int depth,
++	       isccc_sexpr_t **valuep) {
+ 	unsigned int msgtype;
+ 	uint32_t len;
+ 	isccc_sexpr_t *value;
+ 	isccc_region_t active;
+ 	isc_result_t result;
+ 
++	if (depth > ISCCC_MAXDEPTH) {
++		return (ISCCC_R_MAXDEPTH);
++	}
++
+ 	if (REGION_SIZE(*source) < 1 + 4) {
+ 		return (ISC_R_UNEXPECTEDEND);
+ 	}
+@@ -513,9 +523,9 @@ value_fromwire(isccc_region_t *source, isccc_sexpr_t **valuep) {
+ 			result = ISC_R_NOMEMORY;
+ 		}
+ 	} else if (msgtype == ISCCC_CCMSGTYPE_TABLE) {
+-		result = table_fromwire(&active, NULL, 0, valuep);
++		result = table_fromwire(&active, NULL, 0, depth + 1, valuep);
+ 	} else if (msgtype == ISCCC_CCMSGTYPE_LIST) {
+-		result = list_fromwire(&active, valuep);
++		result = list_fromwire(&active, depth + 1, valuep);
+ 	} else {
+ 		result = ISCCC_R_SYNTAX;
+ 	}
+@@ -525,7 +535,7 @@ value_fromwire(isccc_region_t *source, isccc_sexpr_t **valuep) {
+ 
+ static isc_result_t
+ table_fromwire(isccc_region_t *source, isccc_region_t *secret,
+-	       uint32_t algorithm, isccc_sexpr_t **alistp) {
++	       uint32_t algorithm, unsigned int depth, isccc_sexpr_t **alistp) {
+ 	char key[256];
+ 	uint32_t len;
+ 	isc_result_t result;
+@@ -535,6 +545,10 @@ table_fromwire(isccc_region_t *source, isccc_region_t *secret,
+ 
+ 	REQUIRE(alistp != NULL && *alistp == NULL);
+ 
++	if (depth > ISCCC_MAXDEPTH) {
++		return (ISCCC_R_MAXDEPTH);
++	}
++
+ 	checksum_rstart = NULL;
+ 	first_tag = true;
+ 	alist = isccc_alist_create();
+@@ -551,7 +565,7 @@ table_fromwire(isccc_region_t *source, isccc_region_t *secret,
+ 		GET_MEM(key, len, source->rstart);
+ 		key[len] = '\0'; /* Ensure NUL termination. */
+ 		value = NULL;
+-		result = value_fromwire(source, &value);
++		result = value_fromwire(source, depth + 1, &value);
+ 		if (result != ISC_R_SUCCESS) {
+ 			goto bad;
+ 		}
+@@ -589,14 +603,19 @@ bad:
+ }
+ 
+ static isc_result_t
+-list_fromwire(isccc_region_t *source, isccc_sexpr_t **listp) {
++list_fromwire(isccc_region_t *source, unsigned int depth,
++	      isccc_sexpr_t **listp) {
+ 	isccc_sexpr_t *list, *value;
+ 	isc_result_t result;
+ 
++	if (depth > ISCCC_MAXDEPTH) {
++		return (ISCCC_R_MAXDEPTH);
++	}
++
+ 	list = NULL;
+ 	while (!REGION_EMPTY(*source)) {
+ 		value = NULL;
+-		result = value_fromwire(source, &value);
++		result = value_fromwire(source, depth + 1, &value);
+ 		if (result != ISC_R_SUCCESS) {
+ 			isccc_sexpr_free(&list);
+ 			return (result);
+@@ -628,7 +647,7 @@ isccc_cc_fromwire(isccc_region_t *source, isccc_sexpr_t **alistp,
+ 		return (ISCCC_R_UNKNOWNVERSION);
+ 	}
+ 
+-	return (table_fromwire(source, secret, algorithm, alistp));
++	return (table_fromwire(source, secret, algorithm, 0, alistp));
+ }
+ 
+ static isc_result_t
+diff --git a/lib/isccc/include/isccc/result.h b/lib/isccc/include/isccc/result.h
+index 5346babefc..5b6a876d1c 100644
+--- a/lib/isccc/include/isccc/result.h
++++ b/lib/isccc/include/isccc/result.h
+@@ -46,8 +46,10 @@
+ #define ISCCC_R_CLOCKSKEW (ISC_RESULTCLASS_ISCCC + 4)
+ /*% Duplicate */
+ #define ISCCC_R_DUPLICATE (ISC_RESULTCLASS_ISCCC + 5)
++/*% Maximum recursion depth */
++#define ISCCC_R_MAXDEPTH (ISC_RESULTCLASS_ISCCC + 6)
+ 
+-#define ISCCC_R_NRESULTS 6 /*%< Number of results */
++#define ISCCC_R_NRESULTS 7 /*%< Number of results */
+ 
+ ISC_LANG_BEGINDECLS
+ 
+diff --git a/lib/isccc/result.c b/lib/isccc/result.c
+index 9285435209..1956cb1655 100644
+--- a/lib/isccc/result.c
++++ b/lib/isccc/result.c
+@@ -36,12 +36,14 @@ static const char *text[ISCCC_R_NRESULTS] = {
+ 	"bad auth",	   /* 3 */
+ 	"expired",	   /* 4 */
+ 	"clock skew",	   /* 5 */
+-	"duplicate"	   /* 6 */
++	"duplicate",	   /* 6 */
++	"max depth"	   /* 7 */
+ };
+ 
+ static const char *ids[ISCCC_R_NRESULTS] = {
+ 	"ISCCC_R_UNKNOWNVERSION", "ISCCC_R_SYNTAX",    "ISCCC_R_BADAUTH",
+ 	"ISCCC_R_EXPIRED",	  "ISCCC_R_CLOCKSKEW", "ISCCC_R_DUPLICATE",
++	"ISCCC_R_MAXDEPTH"
+ };
+ 
+ #define ISCCC_RESULT_RESULTSET 2
+-- 
+2.41.0
+

bind.spec

@@ -51,7 +51,7 @@ Summary:  The Berkeley Internet Name Domain (BIND) DNS (Domain Name System) serv
 Name:     bind
 License:  MPLv2.0
 Version:  9.16.23
-Release:  13%{?dist}
+Release:  14%{?dist}
 Epoch:    32
 Url:      https://www.isc.org/downloads/bind/
 #
@@ -132,6 +132,7 @@ Patch189: bind-9.16-CVE-2023-2911-1.patch
 Patch190: bind-9.16-CVE-2023-2911-2.patch
 # https://gitlab.isc.org/isc-projects/bind9/commit/c003c5bc3c68f3e513654b6689e1f60280d14844
 Patch191: bind-9.16-CVE-2023-2911-3.patch
+Patch192: bind-9.16-CVE-2023-3341.patch
 
 %{?systemd_ordering}
 Requires:       coreutils
@@ -451,6 +452,7 @@ in HTML and PDF format.
 %patch189 -p1 -b .CVE-2023-2911-1
 %patch190 -p1 -b .CVE-2023-2911-2
 %patch191 -p1 -b .CVE-2023-2911-3
+%patch192 -p1 -b .CVE-2023-3341
 
 %if %{with PKCS11}
 %patch135 -p1 -b .config-pkcs11
@@ -1174,6 +1176,9 @@ fi;
 %endif
 
 %changelog
+* Wed Sep 20 2023 Petr Menšík <pemensik@redhat.com> - 32:9.16.23-14
+- Limit the amount of recursion possible in control channel (CVE-2023-3341)
+
 * Mon Jun 26 2023 Petr Menšík <pemensik@redhat.com> - 32:9.16.23-13
 - Prevent possible endless loop when refreshing stale data (CVE-2023-2911)