more clang-tidy

diff --git a/pdns/gss_context.hh b/pdns/gss_context.hh
index ba2e545e92bcfcf2491ec44a7b93f40c031c5b1f..f7fbcef333c91be11886bd117a26f6bbb58bf2f8 100644 (file)
--- a/pdns/gss_context.hh
+++ b/pdns/gss_context.hh
@@ -160,7 +160,7 @@ public:
 
 private:
 #ifdef ENABLE_GSS_TSIG
-  OM_uint32 d_maj, d_min;
+  OM_uint32 d_maj{0}, d_min{0};
   gss_name_t d_name;
 #endif
 }; // GssName

diff --git a/pdns/rfc2136handler.cc b/pdns/rfc2136handler.cc
index 4e30944e0f2ef6a4b7c82435e07405c580e71182..f0d6e9361a6b1e2c7d5a6a906b5eb4b9bcbe6a4d 100644 (file)
--- a/pdns/rfc2136handler.cc
+++ b/pdns/rfc2136handler.cc
@@ -731,8 +731,9 @@ int PacketHandler::processUpdate(DNSPacket& packet) { // NOLINT(readability-func
       return RCode::Refused;
     }
 
-    if (tsigKeys.size() == 0 && packet.d_havetsig)
+    if (tsigKeys.empty() && packet.d_havetsig) {
       g_log<<Logger::Warning<<msgPrefix<<"TSIG is provided, but domain is not secured with TSIG. Processing continues"<<endl;
+    }
 
   }
 
@@ -757,7 +758,7 @@ int PacketHandler::processUpdate(DNSPacket& packet) { // NOLINT(readability-func
 
   DomainInfo di;
   di.backend=nullptr;
-  if(!B.getDomainInfo(packet.qdomain, di) || !di.backend) {
+  if(!B.getDomainInfo(packet.qdomain, di) || (di.backend == nullptr)) {
     g_log<<Logger::Error<<msgPrefix<<"Can't determine backend for domain '"<<packet.qdomain<<"' (or backend does not support DNS update operation)"<<endl;
     return RCode::NotAuth;
   }
@@ -904,7 +905,7 @@ int PacketHandler::processUpdate(DNSPacket& packet) { // NOLINT(readability-func
       if (rr->d_place == DNSResourceRecord::AUTHORITY) {
         /* see if it's permitted by policy */
         if (this->d_update_policy_lua != nullptr) {
-          if (this->d_update_policy_lua->updatePolicy(rr->d_name, QType(rr->d_type), di.zone, packet) == false) {
+          if (!this->d_update_policy_lua->updatePolicy(rr->d_name, QType(rr->d_type), di.zone, packet)) {
             g_log<<Logger::Warning<<msgPrefix<<"Refusing update for " << rr->d_name << "/" << QType(rr->d_type).toString() << ": Not permitted by policy"<<endl;
             continue;
           } else {