dnsdist: Fix clang-tidy's warnings

diff --git a/pdns/dnsdistdist/dnsdist-lua-ffi-interface.h b/pdns/dnsdistdist/dnsdist-lua-ffi-interface.h
index 9a1b7e4cb2b4c1fe3141d278d4c2a6319072a8fd..da6197891c3521deb8cae37a0230c3993d310830 100644 (file)
--- a/pdns/dnsdistdist/dnsdist-lua-ffi-interface.h
+++ b/pdns/dnsdistdist/dnsdist-lua-ffi-interface.h
@@ -83,9 +83,9 @@ bool dnsdist_ffi_dnsquestion_get_ecs_override(const dnsdist_ffi_dnsquestion_t* d
 uint16_t dnsdist_ffi_dnsquestion_get_ecs_prefix_length(const dnsdist_ffi_dnsquestion_t* dq) __attribute__ ((visibility ("default")));
 bool dnsdist_ffi_dnsquestion_is_temp_failure_ttl_set(const dnsdist_ffi_dnsquestion_t* dq) __attribute__ ((visibility ("default")));
 uint32_t dnsdist_ffi_dnsquestion_get_temp_failure_ttl(const dnsdist_ffi_dnsquestion_t* dq) __attribute__ ((visibility ("default")));
-bool dnsdist_ffi_dnsquestion_get_do(const dnsdist_ffi_dnsquestion_t* dq) __attribute__ ((visibility ("default")));
-uint8_t dnsdist_ffi_dnsquestion_get_edns_version(const dnsdist_ffi_dnsquestion_t* dq) __attribute__ ((visibility ("default")));
-uint8_t dnsdist_ffi_dnsquestion_get_edns_extended_rcode(const dnsdist_ffi_dnsquestion_t* dq) __attribute__ ((visibility ("default")));
+bool dnsdist_ffi_dnsquestion_get_do(const dnsdist_ffi_dnsquestion_t* dnsQuestion) __attribute__ ((visibility ("default")));
+uint8_t dnsdist_ffi_dnsquestion_get_edns_version(const dnsdist_ffi_dnsquestion_t* dnsQuestion) __attribute__ ((visibility ("default")));
+uint8_t dnsdist_ffi_dnsquestion_get_edns_extended_rcode(const dnsdist_ffi_dnsquestion_t* dnsQuestion) __attribute__ ((visibility ("default")));
 void dnsdist_ffi_dnsquestion_get_sni(const dnsdist_ffi_dnsquestion_t* dq, const char** sni, size_t* sniSize) __attribute__ ((visibility ("default")));
 const char* dnsdist_ffi_dnsquestion_get_tag(const dnsdist_ffi_dnsquestion_t* dq, const char* label) __attribute__ ((visibility ("default")));
 size_t dnsdist_ffi_dnsquestion_get_tag_raw(const dnsdist_ffi_dnsquestion_t* dq, const char* label, char* buffer, size_t bufferSize) __attribute__ ((visibility ("default")));

diff --git a/pdns/dnsdistdist/dnsdist-lua-ffi.cc b/pdns/dnsdistdist/dnsdist-lua-ffi.cc
index 617fb9ce9534cf928e246aa968111be4b1818057..b54a069010cc4ab2c6ace7f4ecc4c3bcd5ab7b25 100644 (file)
--- a/pdns/dnsdistdist/dnsdist-lua-ffi.cc
+++ b/pdns/dnsdistdist/dnsdist-lua-ffi.cc
@@ -235,20 +235,20 @@ uint32_t dnsdist_ffi_dnsquestion_get_temp_failure_ttl(const dnsdist_ffi_dnsquest
   return 0;
 }
 
-bool dnsdist_ffi_dnsquestion_get_do(const dnsdist_ffi_dnsquestion_t* dq)
+bool dnsdist_ffi_dnsquestion_get_do(const dnsdist_ffi_dnsquestion_t* dnsQuestion)
 {
-  return dnsdist::getEDNSZ(*dq->dq) & EDNS_HEADER_FLAG_DO;
+  return (dnsdist::getEDNSZ(*dnsQuestion->dq) & EDNS_HEADER_FLAG_DO) != 0;
 }
 
-uint8_t dnsdist_ffi_dnsquestion_get_edns_version(const dnsdist_ffi_dnsquestion_t* dq)
+uint8_t dnsdist_ffi_dnsquestion_get_edns_version(const dnsdist_ffi_dnsquestion_t* dnsQuestion)
 {
-  auto version = dnsdist::getEDNSVersion(*dq->dq);
+  auto version = dnsdist::getEDNSVersion(*dnsQuestion->dq);
   return version ? *version : 0U;
 }
 
-uint8_t dnsdist_ffi_dnsquestion_get_edns_extended_rcode(const dnsdist_ffi_dnsquestion_t* dq)
+uint8_t dnsdist_ffi_dnsquestion_get_edns_extended_rcode(const dnsdist_ffi_dnsquestion_t* dnsQuestion)
 {
-  auto rcode = dnsdist::getEDNSExtendedRCode(*dq->dq);
+  auto rcode = dnsdist::getEDNSExtendedRCode(*dnsQuestion->dq);
   return rcode ? *rcode : 0U;
 }
 

diff --git a/pdns/dnsdistdist/test-dnsdist_cc.cc b/pdns/dnsdistdist/test-dnsdist_cc.cc
index 44a3cf75ee66aa5c1cdf963bd33fa00f0eef9aa6..1cca05f66c40ae82e7a953f4eef4c152cb247902 100644 (file)
--- a/pdns/dnsdistdist/test-dnsdist_cc.cc
+++ b/pdns/dnsdistdist/test-dnsdist_cc.cc
@@ -1600,7 +1600,7 @@ BOOST_AUTO_TEST_CASE(test_getEDNSVersion)
   const uint16_t qclass = QClass::IN;
   const GenericDNSPacketWriter<PacketBuffer>::optvect_t opts;
 
-  auto getVersion = [&qname, qtype, qclass](PacketBuffer& query) {
+  auto getVersion = [&qname](PacketBuffer& query) {
     InternalQueryState ids;
     ids.protocol = dnsdist::Protocol::DoUDP;
     ids.qname = qname;
@@ -1662,7 +1662,7 @@ BOOST_AUTO_TEST_CASE(test_getEDNSExtendedRCode)
   const uint16_t qtype = QType::A;
   const uint16_t qclass = QClass::IN;
 
-  auto getExtendedRCode = [&qname, qtype, qclass](PacketBuffer& query) {
+  auto getExtendedRCode = [&qname](PacketBuffer& query) {
     InternalQueryState ids;
     ids.protocol = dnsdist::Protocol::DoUDP;
     ids.qname = qname;