From: Kees Monshouwer <mind04@monshouwer.org>
Date: Sun, 29 Sep 2013 11:45:45 +0000 (+0200)
Subject: pdnssec check-zone, check for missing NS at apex and records in unlikely places
X-Git-Tag: rec-3.6.0-rc1~430^2~3
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=1e2e9565255a42fe9618fd613d71003361f07e96;p=pdns

pdnssec check-zone, check for missing NS at apex and records in unlikely places
---

diff --git a/pdns/pdnssec.cc b/pdns/pdnssec.cc
index 6cfc259a1..e585a5b97 100644
--- a/pdns/pdnssec.cc
+++ b/pdns/pdnssec.cc
@@ -343,6 +343,8 @@ int checkZone(DNSSECKeeper &dk, UeberBackend &B, const std::string& zone)
   DNSResourceRecord rr;
   uint64_t numrecords=0, numerrors=0, numwarnings=0;
 
+
+  bool hasNsAtApex = false;
   set<string> records, cnames, noncnames;
   map<string, unsigned int> ttl;
 
@@ -386,6 +388,24 @@ int checkZone(DNSSECKeeper &dk, UeberBackend &B, const std::string& zone)
       continue;
     }
 
+    if(pdns_iequals(rr.qname, zone)) {
+      if (rr.qtype.getCode() == QType::NS) {
+        hasNsAtApex=true;
+      } else if (rr.qtype.getCode() == QType::DS) {
+        cout<<"[Warning] DS at apex in zone '"<<zone<<"', should no be here."<<endl;
+        numwarnings++;
+      }
+    } else {
+      if (rr.qtype.getCode() == QType::SOA) {
+        cout<<"[Error] SOA record not at apex '"<<rr.qname<<" IN "<<rr.qtype.getName()<<" "<<rr.content<<"' in zone '"<<zone<<"'"<<endl;
+        numerrors++;
+        continue;
+      } else if (rr.qtype.getCode() == QType::DNSKEY) {
+        cout<<"[Warning] DNSKEY record not at apex '"<<rr.qname<<" IN "<<rr.qtype.getName()<<" "<<rr.content<<"' in zone '"<<zone<<"', should not be here."<<endl;
+        numwarnings++;
+      }
+    }
+
     if (rr.qtype.getCode() == QType::CNAME) {
       if (!cnames.count(toLower(rr.qname)))
         cnames.insert(toLower(rr.qname));
@@ -485,6 +505,11 @@ int checkZone(DNSSECKeeper &dk, UeberBackend &B, const std::string& zone)
     }
   }
 
+  if(!hasNsAtApex) {
+    cout<<"[Error] No NS record at zone apex in zone '"<<zone<<"'"<<endl;
+    numerrors++;
+  }
+
   cout<<"Checked "<<numrecords<<" records of '"<<zone<<"', "<<numerrors<<" errors, "<<numwarnings<<" warnings."<<endl;
   return numerrors;
 }