From 0eb07f3a5c23cc94c95a2bed022367085f8dfcef Mon Sep 17 00:00:00 2001
From: Otto Moerbeek <otto.moerbeek@open-xchange.com>
Date: Fri, 20 Dec 2019 10:08:50 +0100
Subject: [PATCH] QName Minimization consults the cache first to see work needs
 to be done. Via doCacheCheck() this can lead to a call to computeZoneCuts()
 which might/does not work properly on only cached records, so always make
 sure it can go out if needed. Should fix #8642.

---
 pdns/syncres.cc | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/pdns/syncres.cc b/pdns/syncres.cc
index e9371524c1..4e031a002a 100644
--- a/pdns/syncres.cc
+++ b/pdns/syncres.cc
@@ -2134,12 +2134,15 @@ void SyncRes::computeZoneCuts(const DNSName& begin, const DNSName& end, unsigned
     return;
   }
 
+  const bool oldCacheOnly = setCacheOnly(false);
+
   dsmap_t ds;
   vState cutState = getDSRecords(end, ds, false, depth);
   LOG(d_prefix<<": setting cut state for "<<end<<" to "<<vStates[cutState]<<endl);
   d_cutStates[end] = cutState;
 
   if (!shouldValidate()) {
+    setCacheOnly(oldCacheOnly);
     return;
   }
 
@@ -2212,6 +2215,7 @@ void SyncRes::computeZoneCuts(const DNSName& begin, const DNSName& end, unsigned
       LOG(" - "<<cut.first<<": "<<vStates[cut.second]<<endl);
     }
   }
+  setCacheOnly(oldCacheOnly);
 }
 
 vState SyncRes::validateDNSKeys(const DNSName& zone, const std::vector<DNSRecord>& dnskeys, const std::vector<std::shared_ptr<RRSIGRecordContent> >& signatures, unsigned int depth)
-- 
2.47.2