more clang-tidy

PowerDNS · Mar 25, 2024 · f831a2e · f831a2e
1 parent 4ec1794
commit f831a2e
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 4 deletions.
diff --git 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
@@ -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 {