From: Bert Hubert <bert.hubert@netherlabs.nl>
Date: Tue, 5 Jul 2005 15:38:50 +0000 (+0000)
Subject: kai's fixes (bsdisms, min/max, bogus)
X-Git-Tag: pdns-2.9.18~17
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=da327a1b4f7b500709a28785e391546696bc7982;p=thirdparty%2Fpdns.git

kai's fixes (bsdisms, min/max, bogus)


git-svn-id: svn://svn.powerdns.com/pdns/trunk/pdns@432 d19b8d6e-7fed-0310-83ef-9ca221ded41b
---

diff --git a/pdns/dnsscope.cc b/pdns/dnsscope.cc
index 47d075c285..a1dbf36f76 100644
--- a/pdns/dnsscope.cc
+++ b/pdns/dnsscope.cc
@@ -1,3 +1,4 @@
+#define __FAVOR_BSD
 #include <pcap.h>
 
 #include "statbag.hh"
@@ -8,6 +9,7 @@
 #include <map>
 #include <set>
 #include <fstream>
+#include <algorithm>
 #include "anadns.hh"
 
 using namespace boost;
@@ -41,7 +43,7 @@ try
   if(argc==3)
     pw=new PcapPacketWriter(argv[2], pr);
 
-  int dnserrors=0;
+  int dnserrors=0, bogus=0;
   typedef map<uint32_t,uint32_t> cumul_t;
   cumul_t cumul;
   unsigned int untracked=0, errorresult=0, reallylate=0;
@@ -53,8 +55,8 @@ try
 
 
   while(pr.getUDPPacket()) {
-    if((ntohs(pr.d_udp->dest)==5300 || ntohs(pr.d_udp->source)==5300 ||
-	ntohs(pr.d_udp->dest)==53   || ntohs(pr.d_udp->source)==53) &&
+    if((ntohs(pr.d_udp->uh_dport)==5300 || ntohs(pr.d_udp->uh_sport)==5300 ||
+	ntohs(pr.d_udp->uh_dport)==53   || ntohs(pr.d_udp->uh_sport)==53) &&
         pr.d_len > sizeof(HEADER)) {
       try {
 	MOADNSParser mdp((const char*)pr.d_payload, pr.d_len);
@@ -111,9 +113,9 @@ try
 	continue;
       }
       catch(exception& e) {
-	cerr<<"Bogus packet"<<endl;
 	if(pw)
 	  pw->write();
+	bogus++;
 	continue;
       }
     }
@@ -121,7 +123,7 @@ try
   cerr<<"Timespan: "<<(highestTime-lowestTime)/3600.0<<" hours"<<endl;
 
   cerr<<"Saw "<<pr.d_correctpackets<<" correct packets, "<<pr.d_runts<<" runts, "<< pr.d_oversized<<" oversize, "<<
-    pr.d_nonetheripudp<<" unknown encaps, "<<dnserrors<<" dns decoding errors"<<endl;
+    pr.d_nonetheripudp<<" unknown encaps, "<<dnserrors<<" dns decoding errors, "<<bogus<<" bogus packets"<<endl;
 
   unsigned int unanswered=0;
   for(statmap_t::const_iterator i=statmap.begin(); i!=statmap.end(); ++i) {