]> granicus.if.org Git - pdns/commitdiff
ixfrutil: ensure it does not print by itself
authorPieter Lexis <pieter.lexis@powerdns.com>
Thu, 12 Apr 2018 15:15:31 +0000 (17:15 +0200)
committerPieter Lexis <pieter.lexis@powerdns.com>
Fri, 13 Apr 2018 08:51:02 +0000 (10:51 +0200)
pdns/ixfrutils.cc
pdns/ixplore.cc

index d7d0cd6173958662a8dfceb7d3ed718426463b0c..6957e9d64710878ab7c4b3b60e3bcb53d1b632e1 100644 (file)
@@ -139,11 +139,7 @@ void loadZoneFromDisk(records_t& records, const string& fname, const DNSName& zo
       seenSOA=true;
     }
   }
-  cout<<"Parsed "<<nrecords<<" records"<<endl;
-  if(rr.qtype.getCode() == QType::SOA && seenSOA) {
-    cout<<"Zone was complete (SOA at end)"<<endl;
-  }
-  else  {
+  if(!(rr.qtype.getCode() == QType::SOA && seenSOA)) {
     records.clear();
     throw runtime_error("Zone not complete!");
   }
index b06eaf0f3d89c5fab33a1bbbc76359457d672d61..3ab49991797154851f461b81cb6f6f6fb60747a5 100644 (file)
@@ -76,8 +76,12 @@ int main(int argc, char** argv) {
       DNSName zone(argv[2]);
       cout<<"Loading before from "<<argv[3]<<endl;
       loadZoneFromDisk(before, argv[3], zone);
+      cout<<"Parsed "<<before.size()<<" records"<<endl;
+      cout<<"Zone was complete (SOA at end)"<<endl;
       cout<<"Loading after from "<<argv[4]<<endl;
       loadZoneFromDisk(after, argv[4], zone);
+      cout<<"Parsed "<<after.size()<<" records"<<endl;
+      cout<<"Zone was complete (SOA at end)"<<endl;
 
       vector<DNSRecord> diff;
 
@@ -133,6 +137,8 @@ int main(int argc, char** argv) {
       string fname=directory+"/"+std::to_string(ourSerial);
       cout<<"Loading serial number "<<ourSerial<<" from file "<<fname<<endl;
       loadZoneFromDisk(records, fname, zone);
+      cout<<"Parsed "<<records.size()<<" records"<<endl;
+      cout<<"Zone was complete (SOA at end)"<<endl;
     }
     catch(std::exception& e) {
       cout<<"Could not load zone from disk: "<<e.what()<<endl;