]> granicus.if.org Git - pdns/commitdiff
Merge pull request #7289 from chbruyand/static-cppcheck
authorPieter Lexis <pieterlexis@users.noreply.github.com>
Wed, 9 Jan 2019 10:59:59 +0000 (11:59 +0100)
committerGitHub <noreply@github.com>
Wed, 9 Jan 2019 10:59:59 +0000 (11:59 +0100)
Fix minor warnings reported by the compiler or cppcheck

1  2 
pdns/dnsdist.cc
pdns/dnsdist.hh
pdns/tcpreceiver.cc
pdns/ws-auth.cc

diff --cc pdns/dnsdist.cc
Simple merge
diff --cc pdns/dnsdist.hh
Simple merge
index 7c7fec0aa62b9dc993e8e248987ab24b74063977,cf573c578876d33a7b58d4f4ee777ca2f32383a9..706305c87aa718be343b0308dee1ce78b9383ade
@@@ -951,17 -951,15 +951,17 @@@ int TCPNameserver::doAXFR(const DNSNam
        for(nsecxrepo_t::const_iterator iter = nsecxrepo.begin(); iter != nsecxrepo.end(); ++iter) {
          if(iter->second.d_auth) {
            NSEC3RecordContent n3rc;
 -          n3rc.d_set = iter->second.d_set;
 -          if (n3rc.d_set.size() && (n3rc.d_set.size() != 1 || !n3rc.d_set.count(QType::NS)))
 -            n3rc.d_set.insert(QType::RRSIG);
 -          n3rc.d_salt=ns3pr.d_salt;
 +          n3rc.set(iter->second.d_set);
 +          const auto numberOfTypesSet = n3rc.numberOfTypesSet();
 +          if (numberOfTypesSet != 0 && (numberOfTypesSet != 1 || !n3rc.isSet(QType::NS))) {
 +            n3rc.set(QType::RRSIG);
 +          }
 +          n3rc.d_salt = ns3pr.d_salt;
            n3rc.d_flags = ns3pr.d_flags;
            n3rc.d_iterations = ns3pr.d_iterations;
 -          n3rc.d_algorithm = 1; // SHA1, fixed in PowerDNS for now
 +          n3rc.d_algorithm = DNSSECKeeper::SHA1; // SHA1, fixed in PowerDNS for now
            nsecxrepo_t::const_iterator inext = iter;
-           inext++;
+           ++inext;
            if(inext == nsecxrepo.end())
              inext = nsecxrepo.begin();
            while(!inext->second.d_auth && inext != iter)
diff --cc pdns/ws-auth.cc
Simple merge