]> git.ipfire.org Git - thirdparty/pdns.git/commitdiff
dnsdist: Fix CodeQL warnings
authorRemi Gacogne <remi.gacogne@powerdns.com>
Fri, 25 Jul 2025 10:05:12 +0000 (12:05 +0200)
committerRemi Gacogne <remi.gacogne@powerdns.com>
Mon, 6 Oct 2025 14:50:25 +0000 (16:50 +0200)
Signed-off-by: Remi Gacogne <remi.gacogne@powerdns.com>
pdns/dnsdistdist/dnsdist-lbpolicies.cc
pdns/dnsdistdist/dnsdist-lua.cc

index 4cdc57b023cc1aa111f5a5bf65e7d0d5b179e54f..56c6fcda20854851bcedb798c8edd33899cabed6 100644 (file)
@@ -345,7 +345,7 @@ void setPoolPolicy(const string& poolName, std::shared_ptr<ServerPolicy> policy)
   }
 
   dnsdist::configuration::updateRuntimeConfiguration([&poolName, &policy](dnsdist::configuration::RuntimeConfiguration& config) {
-    auto [poolIt, inserted] = config.d_pools.emplace(poolName, ServerPool());
+    auto [poolIt, _] = config.d_pools.emplace(poolName, ServerPool());
     poolIt->second.policy = std::move(policy);
   });
 }
@@ -359,7 +359,7 @@ void addServerToPool(const string& poolName, std::shared_ptr<DownstreamState> se
   }
 
   dnsdist::configuration::updateRuntimeConfiguration([&poolName, &server](dnsdist::configuration::RuntimeConfiguration& config) {
-    auto [poolIt, inserted] = config.d_pools.emplace(poolName, ServerPool());
+    auto [poolIt, _] = config.d_pools.emplace(poolName, ServerPool());
     poolIt->second.addServer(server);
   });
 }
@@ -374,7 +374,7 @@ void removeServerFromPool(const string& poolName, std::shared_ptr<DownstreamStat
   }
 
   dnsdist::configuration::updateRuntimeConfiguration([&poolName, &server](dnsdist::configuration::RuntimeConfiguration& config) {
-    auto [poolIt, inserted] = config.d_pools.emplace(poolName, ServerPool());
+    auto [poolIt, _] = config.d_pools.emplace(poolName, ServerPool());
     poolIt->second.removeServer(server);
   });
 }
index f84fc4af61acedefa6093b9bee0daf9344141132..422dcf4032332d94a4edc90c1f0d483efe9646c1 100644 (file)
@@ -1682,7 +1682,7 @@ static void setupLuaConfig(LuaContext& luaCtx, bool client, bool configCheck)
     }
     bool created = false;
     dnsdist::configuration::updateRuntimeConfiguration([&poolName, &created](dnsdist::configuration::RuntimeConfiguration& config) {
-      auto [poolIt, inserted] = config.d_pools.emplace(poolName, ServerPool());
+      auto [_, inserted] = config.d_pools.emplace(poolName, ServerPool());
       created = inserted;
     });