From: Evan Hunt Date: Fri, 2 Mar 2018 19:30:02 +0000 (-0800) Subject: temporarily revert change #4859 X-Git-Tag: v9.13.0~124^2 X-Git-Url: http://git.ipfire.org/gitweb.cgi?a=commitdiff_plain;h=b291b45312478af5224fadfcd3a0aeab7ba80013;p=thirdparty%2Fbind9.git temporarily revert change #4859 --- diff --git a/CHANGES b/CHANGES index 85d4d32c185..f1c4a20f8e8 100644 --- a/CHANGES +++ b/CHANGES @@ -145,11 +145,7 @@ 4860. [bug] isc_int8_t should be signed char. [RT #46973] -4859. [bug] A loop was possible when attempting to validate - unsigned CNAME responses from secure zones; - this caused a delay in returning SERVFAIL and - also increased the chances of encountering - CVE-2017-3145. [RT #46839] +4859. [placeholder] 4858. [security] Addresses could be referenced after being freed in resolver.c, causing an assertion failure. diff --git a/bin/named/bind9.xsl.h b/bin/named/bind9.xsl.h index 1dbf09bcc66..50fd2ba3f24 100644 --- a/bin/named/bind9.xsl.h +++ b/bin/named/bind9.xsl.h @@ -5,11 +5,14 @@ static char xslmsg[] = "\n" "\n" "\n" "\n" diff --git a/lib/dns/validator.c b/lib/dns/validator.c index 212524e9af7..e898576d387 100644 --- a/lib/dns/validator.c +++ b/lib/dns/validator.c @@ -1103,8 +1103,7 @@ check_deadlock(dns_validator_t *val, dns_name_t *name, dns_rdatatype_t type, for (parent = val; parent != NULL; parent = parent->parent) { if (parent->event != NULL && - (parent->event->type == type || - parent->event->type == dns_rdatatype_cname) && + parent->event->type == type && dns_name_equal(parent->event->name, name) && /* * As NSEC3 records are meta data you sometimes