From: Kees Monshouwer <mind04@monshouwer.org>
Date: Mon, 30 Sep 2013 22:04:57 +0000 (+0200)
Subject: pdnssec check-zone, more tolerant IPv6 format check
X-Git-Tag: rec-3.6.0-rc1~424^2
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=0f470360cfed98d057e8ed96f214df45ca798149;p=pdns

pdnssec check-zone, more tolerant IPv6 format check
---

diff --git a/pdns/pdnssec.cc b/pdns/pdnssec.cc
index 365a30ee0..45554d875 100644
--- a/pdns/pdnssec.cc
+++ b/pdns/pdnssec.cc
@@ -378,10 +378,23 @@ int checkZone(DNSSECKeeper &dk, UeberBackend &B, const std::string& zone)
       shared_ptr<DNSRecordContent> drc(DNSRecordContent::mastermake(rr.qtype.getCode(), 1, rr.content));
       string tmp=drc->serialize(rr.qname);
       tmp = drc->getZoneRepresentation();
-      if (!pdns_iequals(tmp, rr.content)) {
-        cout<<"[Warning] Parsed and original record content are not equal: "<<rr.qname<<" IN " <<rr.qtype.getName()<< " '" << rr.content<<"' (Content parsed as '"<<tmp<<"')"<<endl;
+      if (rr.qtype.getCode() != QType::AAAA) {
+        if (!pdns_iequals(tmp, rr.content)) {
+          cout<<"[Warning] Parsed and original record content are not equal: "<<rr.qname<<" IN " <<rr.qtype.getName()<< " '" << rr.content<<"' (Content parsed as '"<<tmp<<"')"<<endl;
+          rr.content=tmp;
+          numwarnings++;
+        }
+      } else {
+        struct addrinfo hint, *res;
+        memset(&hint, 0, sizeof(hint));
+        hint.ai_family = AF_INET6;
+        hint.ai_flags = AI_NUMERICHOST;
+        if(getaddrinfo(rr.content.c_str(), 0, &hint, &res)) {
+          cout<<"[Warning] Folowing record is not a vallid IPv6 address: "<<rr.qname<<" IN " <<rr.qtype.getName()<< " '" << rr.content<<"'"<<endl;
+          numwarnings++;
+        } else
+          freeaddrinfo(res);
         rr.content=tmp;
-        numwarnings++;
       }
     }
     catch(std::exception& e)