dnsdist: Fix clang-tidy warnings

diff --git a/pdns/dnsdistdist/dnsdist-lua-ffi.cc b/pdns/dnsdistdist/dnsdist-lua-ffi.cc
index 1bbb9c2fdfce25e7f5a8425a24bede967a45002b..6af7562efb22488a95f383f8bcac09108adff4fc 100644 (file)
--- a/pdns/dnsdistdist/dnsdist-lua-ffi.cc
+++ b/pdns/dnsdistdist/dnsdist-lua-ffi.cc
@@ -2165,15 +2165,18 @@ bool dnsdist_ffi_dnsquestion_generate_svc_response(dnsdist_ffi_dnsquestion_t* dn
   std::vector<SVCRecordParameters> parameters;
   parameters.reserve(parametersListSize);
   for (size_t idx = 0; idx < parametersListSize; idx++) {
-    if (parametersList[idx] == nullptr) {
+    // NOLINTNEXTLINE(cppcoreguidelines-pro-bounds-pointer-arithmetic): this is a C API
+    const auto& parameter = parametersList[idx];
+    if (parameter == nullptr) {
       return false;
     }
-    parameters.push_back(parametersList[idx]->parameters);
+    parameters.push_back(parameter->parameters);
   }
   return dnsdist::svc::generateSVCResponse(*dnsQuestion->dq, ttl, parameters);
 }
 
 void dnsdist_ffi_svc_record_parameters_free(dnsdist_ffi_svc_record_parameters* parameters)
 {
+  // NOLINTNEXTLINE(cppcoreguidelines-owning-memory): this is a C API, RAII is not an option
   delete parameters;
 }

diff --git a/pdns/dnsdistdist/dnsdist-svc.cc b/pdns/dnsdistdist/dnsdist-svc.cc
index 6f73000ea8c42b358392ccbbf1a87f7d548c083d..5393f400d63fa5b1e8b938e768ceadc3831e3428 100644 (file)
--- a/pdns/dnsdistdist/dnsdist-svc.cc
+++ b/pdns/dnsdistdist/dnsdist-svc.cc
@@ -140,7 +140,7 @@ namespace dnsdist::svc
 bool generateSVCResponse(DNSQuestion& dnsQuestion, const std::vector<std::vector<uint8_t>>& svcRecordPayloads, const std::set<std::pair<DNSName, ComboAddress>>& additionals4, const std::set<std::pair<DNSName, ComboAddress>>& additionals6, const ResponseConfig& responseConfig)
 {
   /* it will likely be a bit bigger than that because of additionals */
-  auto totalPayloadsSize = 0;
+  size_t totalPayloadsSize = 0;
   for (const auto& payload : svcRecordPayloads) {
     totalPayloadsSize += payload.size();
   }