Skip to content

Commit

Permalink
dnsdist: Fix a couple warnings reported by clang++
Browse files Browse the repository at this point in the history
  • Loading branch information
rgacogne committed Jan 20, 2023
1 parent b03958c commit f3d1722
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 3 deletions.
2 changes: 0 additions & 2 deletions pdns/cdb.cc
Original file line number Diff line number Diff line change
Expand Up @@ -145,9 +145,7 @@ vector<string> CDB::findall(string &key)
throw std::runtime_error("Error looking up key '" + key + "' from CDB database: " + std::to_string(res));
}

int x=0;
while(cdb_findnext(&cdbf) > 0) {
x++;
unsigned int vpos = cdb_datapos(&d_cdb);
unsigned int vlen = cdb_datalen(&d_cdb);
std::string val;
Expand Down
2 changes: 1 addition & 1 deletion pdns/dnsdistdist/test-dnsdistrings_cc.cc
Original file line number Diff line number Diff line change
Expand Up @@ -208,7 +208,7 @@ BOOST_AUTO_TEST_CASE(test_Rings_Threaded) {
Rings rings(numberOfEntries, numberOfShards, lockAttempts, true);
rings.init();
#if defined(DNSDIST_RINGS_WITH_MACADDRESS)
Rings::Query query({requestor, qname, now, dh, size, qtype, protocol, "", false});
Rings::Query query({requestor, qname, now, dh, size, qtype, protocol, dnsdist::MacAddress(), false});
#else
Rings::Query query({requestor, qname, now, dh, size, qtype, protocol});
#endif
Expand Down

0 comments on commit f3d1722

Please sign in to comment.