From: Otto Moerbeek <otto.moerbeek@open-xchange.com>
Date: Tue, 15 Mar 2022 12:35:54 +0000 (+0100)
Subject: Process comment reviews
X-Git-Tag: rec-4.7.0-beta1~44^2~5
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=539b506ffdb5ec557e9b9e5bee98e1a2120ad56a;p=thirdparty%2Fpdns.git

Process comment reviews
---

diff --git a/pdns/syncres.cc b/pdns/syncres.cc
index a5bf23b1ce..b60f78d0de 100644
--- a/pdns/syncres.cc
+++ b/pdns/syncres.cc
@@ -3487,7 +3487,7 @@ void SyncRes::rememberParentSetIfNeeded(const DNSName& domain, const vector<DNSR
   for (const auto& ns : existing) {
     auto content = getRR<NSRecordContent>(ns);
     if (authSet.count(content->getNS()) == 0) {
-      LOG(d_prefix << domain << ": at least one child NS was not in the parent NS set, remembering parent NS set and cached IPs" << endl);
+      LOG(d_prefix << domain << ": at least one parent-side NS was not in the child-side NS set, remembering parent NS set and cached IPs" << endl);
       shouldSave = true;
       break;
     }
@@ -4743,7 +4743,7 @@ int SyncRes::doResolveAt(NsSet &nameservers, DNSName auth, bool flawedNSSet, con
           /* if tns is empty, retrieveAddressesForNS() knows we have hardcoded servers (i.e. "forwards") */
           remoteIPs = retrieveAddressesForNS(prefix, qname, tns, depth, beenthere, rnameservers, nameservers, sendRDQuery, pierceDontQuery, flawedNSSet, cacheOnly, addressQueriesForNS);
         } else {
-          // should be save, caller makes sure nameservers and fallback contain the same names
+          // should be safe, caller makes sure nameservers and fallback contain the same names
           remoteIPs = fallBack->at(tns->first);
         }