]> git.ipfire.org Git - thirdparty/pdns.git/commitdiff
Address Coverity reported unnecessary object copies. 16638/head
authorMiod Vallat <miod.vallat@powerdns.com>
Fri, 12 Dec 2025 13:22:01 +0000 (14:22 +0100)
committerMiod Vallat <miod.vallat@powerdns.com>
Fri, 12 Dec 2025 13:22:01 +0000 (14:22 +0100)
Signed-off-by: Miod Vallat <miod.vallat@powerdns.com>
modules/geoipbackend/geoipbackend.cc
pdns/auth-secondarycommunicator.cc

index 0c4f90027fe4a44bfdc722f56cac79a96974e548..998b12ec5631ae878a2168b345092f3dbeb54bfb 100644 (file)
@@ -302,7 +302,7 @@ bool GeoIPBackend::loadDomain(const std::string& origin, const YAML::Node& domai
           rr.weight = 100;
         }
         rr.auth = true;
-        rrs.push_back(rr);
+        rrs.push_back(std::move(rr));
       }
       std::swap(dom.records[qname.operator const DNSName&()], rrs);
     }
index eb17c8550801b756f0d92990f3f10626a19a6df9..b0153eb5bb7a9378d811a4524f96a5d3da754f16 100644 (file)
@@ -412,7 +412,7 @@ static bool catalogProcess(const XFRContext& ctx, vector<DNSResourceRecord>& rrs
           if (content.length() >= 2 && content.at(0) == '\"' && content.at(content.length() - 1) == '\"') { // TXT pain
             content = content.substr(1, content.length() - 2);
           }
-          ci.d_group.insert(content);
+          ci.d_group.insert(std::move(content));
         }
       }
     }
@@ -556,7 +556,7 @@ static bool processRecordForZS(const DNSName& domain, bool& firstNSEC3, DNSResou
     ctx.optOutFlag = static_cast<bool>(ns3rc.d_flags & 1);
     if (ns3rc.isSet(QType::NS) && !(rr.qname == domain)) {
       DNSName hashPart = rr.qname.makeRelative(domain);
-      ctx.secured.insert(hashPart);
+      ctx.secured.insert(std::move(hashPart));
     }
     return false;
   }