prevent query_coveringnsec() from running twice

when synthesizing a new CNAME, we now check whether the target
matches the query already being processed. if so, we do not
restart the query; this prevents a waste of resources.

(cherry picked from commit 0ae8b2e056c3e24df0746541c505bc358352f7fa)

diff --git a/lib/ns/query.c b/lib/ns/query.c
index 5cf502eef006ffce4a90c8f834579371fe3ab208..4503b8d9fe16b9f6c75ce31d29086ad91dbca78a 100644 (file)
--- a/lib/ns/query.c
+++ b/lib/ns/query.c
@@ -9651,6 +9651,12 @@ query_synthcnamewildcard(query_ctx_t *qctx, dns_rdataset_t *rdataset,
        RUNTIME_CHECK(result == ISC_R_SUCCESS);
        dns_rdata_reset(&rdata);
 
+       if (dns_name_equal(qctx->client->query.qname, &cname.cname)) {
+               dns_message_puttempname(qctx->client->message, &tname);
+               dns_rdata_freestruct(&cname);
+               return (ISC_R_SUCCESS);
+       }
+
        dns_name_copynf(&cname.cname, tname);
 
        dns_rdata_freestruct(&cname);