]> git.ipfire.org Git - thirdparty/pdns.git/commitdiff
qla-v6 outgoing: increase chance of catching errors
authorPieter Lexis <pieter.lexis@powerdns.com>
Wed, 10 Jun 2020 12:34:48 +0000 (14:34 +0200)
committerOtto Moerbeek <otto.moerbeek@open-xchange.com>
Fri, 3 Jul 2020 08:29:09 +0000 (10:29 +0200)
pdns/pdns_recursor.cc
pdns/recursordist/test-syncres_cc10.cc

index 71ea44bb06fe2da3bf3a413aaf3c93230aaa95dc..26439271a6eff24f2ae75952e9089d1190852a90 100644 (file)
@@ -4178,8 +4178,8 @@ static int serviceMain(int argc, char*argv[])
     g_log<<Logger::Warning<<"NOT using IPv6 for outgoing queries - add an IPv6 address (like '::') to query-local-address to enable"<<endl;
   }
 
-  if (!(SyncRes::s_doIPv6 && SyncRes::s_doIPv4)) {
-    g_log<<Logger::Error<<"No outgoing addresses specified! Can not continue"<<endl;
+  if (!SyncRes::s_doIPv6 && !SyncRes::s_doIPv4) {
+    g_log<<Logger::Error<<"No outgoing addresses configured! Can not continue"<<endl;
     exit(99);
   }
 
index 61182c3a99fc308f9042e2fe6367799d47dfad7f..08641de4895568a90000bd5077f22a67e758c0ef 100644 (file)
@@ -19,6 +19,8 @@ BOOST_AUTO_TEST_CASE(test_outgoing_v4_only)
     queries++;
     if (isRootServer(ip)) {
       setLWResult(res, 0, false, false, true);
+      v4Hit |= ip.isIPv4();
+      v6Hit |= ip.isIPv6();
 
       if (domain == DNSName("powerdns.com.")) {
         addRecordToLW(res, domain, QType::NS, "ns1.powerdns.com.", DNSResourceRecord::AUTHORITY, 172800);
@@ -29,7 +31,7 @@ BOOST_AUTO_TEST_CASE(test_outgoing_v4_only)
     }
     else if (ip == ComboAddress("192.0.2.1:53")) {
       setLWResult(res, 0, true, false, false);
-      v4Hit = true;
+      v4Hit |= true;
       if (domain == DNSName("powerdns.com.")) {
         addRecordToLW(res, domain, QType::A, "192.0.2.2");
       }
@@ -37,7 +39,7 @@ BOOST_AUTO_TEST_CASE(test_outgoing_v4_only)
     }
     else if (ip == ComboAddress("[2001:DB8:1::53]:53")) {
       setLWResult(res, 0, true, false, false);
-      v6Hit = true;
+      v6Hit |= true;
       if (domain == DNSName("powerdns.com.")) {
         addRecordToLW(res, domain, QType::A, "192.0.2.2");
       }