permanently disable QNAME minimization in a fetch when forwarding

QNAME minimization is normally disabled when forwarding. if, in the
course of processing a fetch, we switch back to normal recursion at
some point, we can't safely start minimizing because we may have
been left in an inconsistent state.

diff --git a/lib/dns/resolver.c b/lib/dns/resolver.c
index a661ab728398fc24cb559eb04eca768ef9c79823..8956baee8124f1337f82654adda4d12e0e9acaff 100644 (file)
--- a/lib/dns/resolver.c
+++ b/lib/dns/resolver.c
@@ -4023,6 +4023,15 @@ fctx_nextaddress(fetchctx_t *fctx) {
                        addrinfo->flags |= FCTX_ADDRINFO_MARK;
                        fctx->find = NULL;
                        fctx->forwarding = true;
+
+                       /*
+                        * QNAME minimization is disabled when
+                        * forwarding, and has to remain disabled if
+                        * we switch back to normal recursion; otherwise
+                        * forwarding could leave us in an inconsistent
+                        * state.
+                        */
+                       fctx->minimized = false;
                        return (addrinfo);
                }
        }