dnsdist: Avoid copies in for loops

Reported by clang's '-Wrange-loop-construct' since version 10.

diff --git a/pdns/dnscrypt.cc b/pdns/dnscrypt.cc
index 653156a00ade905d589657d0b2c7bc08b9b085af..db096659decac08d5b5c44a10d32b0f7b14c91f3 100644 (file)
--- a/pdns/dnscrypt.cc
+++ b/pdns/dnscrypt.cc
@@ -430,7 +430,7 @@ void DNSCryptContext::getCertificateResponse(time_t now, const DNSName& qname, u
   dh->rcode = RCode::NoError;
 
   ReadLock r(&d_lock);
-  for (const auto pair : d_certs) {
+  for (const auto& pair : d_certs) {
     if (!pair->active || !pair->cert.isValid(now)) {
       continue;
     }

diff --git a/pdns/dnsdist-cache.cc b/pdns/dnsdist-cache.cc
index 2736dc0480ce441d59bf0d1e3facc3b681a2a15d..c97c6744532203e03383895ca67536a9d629987e 100644 (file)
--- a/pdns/dnsdist-cache.cc
+++ b/pdns/dnsdist-cache.cc
@@ -460,7 +460,7 @@ uint64_t DNSDistPacketCache::dump(int fd)
     ReadLock w(&d_shards.at(shardIndex).d_lock);
     auto& map = d_shards[shardIndex].d_map;
 
-    for(const auto entry : map) {
+    for (const auto& entry : map) {
       const CacheValue& value = entry.second;
       count++;
 

diff --git a/pdns/dnsdist-lua-bindings.cc b/pdns/dnsdist-lua-bindings.cc
index 3089936feed4b03bc2b99d2926327f7e2b671371..bc66083f1b226f41cb3b0bf86400f5d67a622143 100644 (file)
--- a/pdns/dnsdist-lua-bindings.cc
+++ b/pdns/dnsdist-lua-bindings.cc
@@ -245,14 +245,14 @@ void setupLuaBindings(bool client)
       }
       if (name.type() == typeid(vector<pair<int, DNSName>>)) {
           auto names = boost::get<vector<pair<int, DNSName>>>(name);
-          for (auto const n : names) {
+          for (const auto& n : names) {
             smn.add(n.second);
           }
           return;
       }
       if (name.type() == typeid(vector<pair<int, string>>)) {
           auto names = boost::get<vector<pair<int, string>>>(name);
-          for (auto const n : names) {
+          for (const auto& n : names) {
             smn.add(n.second);
           }
           return;
@@ -272,14 +272,14 @@ void setupLuaBindings(bool client)
       }
       if (name.type() == typeid(vector<pair<int, DNSName>>)) {
           auto names = boost::get<vector<pair<int, DNSName>>>(name);
-          for (auto const n : names) {
+          for (const auto& n : names) {
             smn.remove(n.second);
           }
           return;
       }
       if (name.type() == typeid(vector<pair<int, string>>)) {
           auto names = boost::get<vector<pair<int, string>>>(name);
-          for (auto const n : names) {
+          for (const auto& n : names) {
             DNSName d(n.second);
             smn.remove(d);
           }

diff --git a/pdns/dnsdist-lua.cc b/pdns/dnsdist-lua.cc
index f2423353ea38d88c0d6ec65430a887aee423fb44..d51156e8ad309a350eea917d3dcdf0e1e94006ac 100644 (file)
--- a/pdns/dnsdist-lua.cc
+++ b/pdns/dnsdist-lua.cc
@@ -113,7 +113,7 @@ static void parseLocalBindVars(boost::optional<localbind_t> vars, bool& reusePor
       interface = boost::get<std::string>((*vars)["interface"]);
     }
     if (vars->count("cpus")) {
-      for (const auto cpu : boost::get<std::vector<std::pair<int,int>>>((*vars)["cpus"])) {
+      for (const auto& cpu : boost::get<std::vector<std::pair<int,int>>>((*vars)["cpus"])) {
         cpus.insert(cpu.second);
       }
     }
@@ -465,7 +465,7 @@ static void setupLuaConfig(bool client, bool configCheck)
       }
 
       if(vars.count("cpus")) {
-        for (const auto cpu : boost::get<vector<pair<int,string>>>(vars["cpus"])) {
+        for (const auto& cpu : boost::get<vector<pair<int,string>>>(vars["cpus"])) {
           cpus.insert(std::stoi(cpu.second));
         }
       }