From: Otto <otto.moerbeek@open-xchange.com>
Date: Tue, 3 Aug 2021 07:08:09 +0000 (+0200)
Subject: Check in more places if the config policy has been updated before
X-Git-Tag: dnsdist-1.7.0-alpha1~71^2
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=refs%2Fpull%2F10627%2Fhead;p=thirdparty%2Fpdns.git

Check in more places if the config policy has been updated before
using or modifying it.
---

diff --git a/pdns/rpzloader.cc b/pdns/rpzloader.cc
index 22b63ef3fa..77cf0926c2 100644
--- a/pdns/rpzloader.cc
+++ b/pdns/rpzloader.cc
@@ -469,7 +469,15 @@ void RPZIXFRTracker(const std::vector<ComboAddress>& primaries, boost::optional<
     try {
       g_log<<Logger::Info<<"Processing "<<deltas.size()<<" delta"<<addS(deltas)<<" for RPZ "<<zoneName<<endl;
 
+      if (luaconfsLocal->generation != configGeneration) {
+        g_log<<Logger::Info<<"A more recent configuration has been found, stopping the existing RPZ update thread for "<<zoneName<<endl;
+        return;
+      }
       oldZone = luaconfsLocal->dfe.getZone(zoneIdx);
+      if (!oldZone || oldZone->getDomain() != zoneName) {
+        g_log<<Logger::Info<<"This policy is no more, stopping the existing RPZ update thread for "<<zoneName << endl;
+        return;
+      }
       /* we need to make a _full copy_ of the zone we are going to work on */
       std::shared_ptr<DNSFilterEngine::Zone> newZone = std::make_shared<DNSFilterEngine::Zone>(*oldZone);
       /* initialize the current serial to the last one */
@@ -540,6 +548,10 @@ void RPZIXFRTracker(const std::vector<ComboAddress>& primaries, boost::optional<
          but we don't want to touch anything else, especially other zones,
          since they might have been updated by another RPZ IXFR tracker thread.
       */
+      if (luaconfsLocal->generation != configGeneration) {
+        g_log<<Logger::Info<<"A more recent configuration has been found, stopping the existing RPZ update thread for "<<zoneName<<endl;
+        return;
+      }
       g_luaconfs.modify([zoneIdx, &newZone](LuaConfigItems& lci) {
                           lci.dfe.setZone(zoneIdx, newZone);
                         });