]> git.ipfire.org Git - thirdparty/bind9.git/commitdiff
Clear DNS_FETCHOPT_TRYSTALE_ONTIMEOUT
authorMark Andrews <marka@isc.org>
Wed, 3 Apr 2024 06:37:14 +0000 (17:37 +1100)
committerNicki Křížek <nicki@isc.org>
Thu, 6 Jun 2024 16:28:35 +0000 (18:28 +0200)
When calling dns_resolver_createfetch in resolver.c with a callback
of resume_dslookup, clear DNS_FETCHOPT_TRYSTALE_ONTIMEOUT from
options as DNS_EVENT_TRYSTALE is not an expected event type and
triggers a REQUIRE.

(cherry picked from commit 6faea6da3d646557d234d63ddd5d524d222e8082)

lib/dns/resolver.c

index 09526246899fce4b4d57ad5fd1c839c47293e9f7..4c621329b87eb9abc00f259eed9b25e2b4e0f598 100644 (file)
@@ -7621,6 +7621,7 @@ resume_dslookup(isc_task_t *task, isc_event_t *event) {
        dns_rdataset_t nameservers;
        dns_fixedname_t fixed;
        dns_name_t *domain;
+       unsigned int options;
 
        REQUIRE(event->ev_type == DNS_EVENT_FETCHDONE);
        fevent = (dns_fetchevent_t *)event;
@@ -7730,9 +7731,10 @@ resume_dslookup(isc_task_t *task, isc_event_t *event) {
 
                FCTXTRACE("continuing to look for parent's NS records");
 
+               options = fctx->options & ~DNS_FETCHOPT_TRYSTALE_ONTIMEOUT;
                result = dns_resolver_createfetch(
                        fctx->res, &fctx->nsname, dns_rdatatype_ns, domain,
-                       nsrdataset, NULL, NULL, 0, fctx->options, 0, NULL, task,
+                       nsrdataset, NULL, NULL, 0, options, 0, NULL, task,
                        resume_dslookup, fctx, &fctx->nsrrset, NULL,
                        &fctx->nsfetch);
                /*
@@ -10059,7 +10061,7 @@ static void
 rctx_chaseds(respctx_t *rctx, dns_message_t *message,
             dns_adbaddrinfo_t *addrinfo, isc_result_t result) {
        fetchctx_t *fctx = rctx->fctx;
-       unsigned int n;
+       unsigned int n, options;
 
        add_bad(fctx, message, addrinfo, result, rctx->broken_type);
        fctx_cancelqueries(fctx, true, false);
@@ -10071,10 +10073,11 @@ rctx_chaseds(respctx_t *rctx, dns_message_t *message,
 
        FCTXTRACE("suspending DS lookup to find parent's NS records");
 
+       options = fctx->options & ~DNS_FETCHOPT_TRYSTALE_ONTIMEOUT;
        result = dns_resolver_createfetch(
                fctx->res, &fctx->nsname, dns_rdatatype_ns, NULL, NULL, NULL,
-               NULL, 0, fctx->options, 0, NULL, rctx->task, resume_dslookup,
-               fctx, &fctx->nsrrset, NULL, &fctx->nsfetch);
+               NULL, 0, options, 0, NULL, rctx->task, resume_dslookup, fctx,
+               &fctx->nsrrset, NULL, &fctx->nsfetch);
        if (result != ISC_R_SUCCESS) {
                if (result == DNS_R_DUPLICATE) {
                        result = DNS_R_SERVFAIL;