From: Bert Hubert <bert.hubert@netherlabs.nl>
Date: Thu, 16 Jan 2003 17:24:36 +0000 (+0000)
Subject: recursor work, alignment issues
X-Git-Tag: pdns-2.9.5~8
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=7b35aa49a8d56d4cb5c57a904ea8a15f44ca637b;p=pdns

recursor work, alignment issues


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

diff --git a/pdns/Makefile.am b/pdns/Makefile.am
index 45edf65a0..8aa16833e 100644
--- a/pdns/Makefile.am
+++ b/pdns/Makefile.am
@@ -35,7 +35,7 @@ backends/bind/bindbackend.cc backends/bind/zoneparser2.cc \
 backends/bind/bindparser.cc backends/bind/bindlexer.c \
 backends/bind/huffman.cc backends/gsql/gsqlbackend.cc \
 backends/gsql/gsqlbackend.hh backends/gsql/ssql.hh \
-sillyrecords.cc recbcomm.hh recbcomm.cc
+sillyrecords.cc 
 
 #
 pdns_server_LDFLAGS= @moduleobjects@ @modulelibs@ @DYNLINKFLAGS@ @LIBDL@ @THREADFLAGS@
diff --git a/pdns/misc.cc b/pdns/misc.cc
index de5ded5b2..70e671fe4 100644
--- a/pdns/misc.cc
+++ b/pdns/misc.cc
@@ -313,10 +313,14 @@ void cleanSlashes(string &str)
   str=out;
 }
 
+
 const string sockAddrToString(struct sockaddr_in *remote, Utility::socklen_t socklen) 
 {    
-  if(socklen==sizeof(struct sockaddr_in))
-     return inet_ntoa(((struct sockaddr_in *)remote)->sin_addr);
+  if(socklen==sizeof(struct sockaddr_in)) {
+    struct sockaddr_in sip;
+    memcpy(&sip,(struct sockaddr_in*)remote,sizeof(sip));
+    return inet_ntoa(sip.sin_addr);
+  }
 #ifdef HAVE_IPV6
   else {
     char tmp[128];
diff --git a/pdns/misc.hh b/pdns/misc.hh
index 8651631a3..d950d319d 100644
--- a/pdns/misc.hh
+++ b/pdns/misc.hh
@@ -69,6 +69,12 @@ inline void putLong(char* p, u_int32_t val)
   putLong((unsigned char *)p,val);
 }
 
+
+inline u_int32_t getLong(unsigned char *p)
+{
+  return (p[0]<<24)+(p[1]<<16)+(p[2]<<8)+p[3];
+}
+
 void upperCase(string& s);
 
 struct ServiceTuple
diff --git a/pdns/pdns_recursor.cc b/pdns/pdns_recursor.cc
index b8801aed7..886681f84 100644
--- a/pdns/pdns_recursor.cc
+++ b/pdns/pdns_recursor.cc
@@ -156,8 +156,8 @@ void startDoResolve(void *p)
     R->setA(false);
     R->setRA(true);
 
-    SyncRes<LWRes> sr;
-    L<<Logger::Error<<"["<<MT.getTid()<<"] new question arrived for '"<<P.qdomain<<"|"<<P.qtype.getName()<<"' from "<<P.getRemote()<<endl;
+    SyncRes sr;
+    L<<Logger::Error<<"["<<MT.getTid()<<"] question for '"<<P.qdomain<<"|"<<P.qtype.getName()<<"' from "<<P.getRemote()<<endl;
     sr.setId(MT.getTid());
     if(!P.d.rd)
       sr.setCacheOnly();
@@ -173,8 +173,8 @@ void startDoResolve(void *p)
 
     const char *buffer=R->getData();
     sendto(d_serversock,buffer,R->len,0,(struct sockaddr *)(R->remote),R->d_socklen);
-    L<<Logger::Error<<"["<<MT.getTid()<<"] sent answer to "<<(P.d.rd?"":"non-rd ")<<"question for '"<<P.qdomain<<"|"<<P.qtype.getName()<<"' to "<<P.getRemote();
-    L<<", "<<ntohs(R->d.ancount)<<" answers, "<<ntohs(R->d.arcount)<<" additional, took "<<sr.d_outqueries<<" packets"<<endl;
+    L<<Logger::Error<<"["<<MT.getTid()<<"] answer to "<<(P.d.rd?"":"non-rd ")<<"question '"<<P.qdomain<<"|"<<P.qtype.getName();
+    L<<"': "<<ntohs(R->d.ancount)<<" answers, "<<ntohs(R->d.arcount)<<" additional, took "<<sr.d_outqueries<<" packets, rcode="<<res<<endl;
     delete R;
   }
   catch(AhuException &ae) {
@@ -263,12 +263,12 @@ void houseKeeping(void *)
     if(qcounter) {
       L<<Logger::Error<<"stats: "<<qcounter<<" questions, "<<cache.size()<<" cache entries, "
        <<(int)((cacheHits*100.0)/(cacheHits+cacheMisses))<<"% cache hits";
-      L<<Logger::Error<<", outpacket/query ratio "<<(int)(SyncRes<LWRes>::s_outqueries*100.0/SyncRes<LWRes>::s_queries)<<"%"<<endl;
+      L<<Logger::Error<<", outpacket/query ratio "<<(int)(SyncRes::s_outqueries*100.0/SyncRes::s_queries)<<"%"<<endl;
     }
     last_stat=time(0);
   }
   if(time(0)-last_rootupdate>7200) {
-    SyncRes<LWRes> sr;
+    SyncRes sr;
     vector<DNSResourceRecord>ret;
 
     sr.setNoCache();
@@ -290,7 +290,7 @@ int main(int argc, char **argv)
     arg().set("soa-serial-offset","0")="0";
     arg().set("local-port","port to listen on")="5300";
     arg().set("local-address","port to listen on")="0.0.0.0";
-    arg().set("trace","if we should output heaps of logging")="true";
+    arg().set("trace","if we should output heaps of logging")="off";
     arg().set("daemon","Operate as a daemon")="no";
 
     arg().parse(argc, argv);
@@ -299,7 +299,7 @@ int main(int argc, char **argv)
     L.toConsole(Logger::Warning);
 
     if(arg().mustDo("trace"))
-      SyncRes<LWRes>::setLog(true);
+      SyncRes::setLog(true);
     
     makeClientSocket();
     makeServerSocket();
diff --git a/pdns/syncres.cc b/pdns/syncres.cc
index 99733fe19..6072a044b 100644
--- a/pdns/syncres.cc
+++ b/pdns/syncres.cc
@@ -30,15 +30,15 @@
 #include "arguments.hh"
 #include "lwres.hh"
 
-template<class MultiPlexor>map<string,string> SyncRes<MultiPlexor>::s_negcache;
-template<class MultiPlexor>unsigned int SyncRes<MultiPlexor>::s_queries;
-template<class MultiPlexor>unsigned int SyncRes<MultiPlexor>::s_outqueries;
-template<class MultiPlexor>bool SyncRes<MultiPlexor>::s_log;
+map<string,string> SyncRes::s_negcache;
+unsigned int SyncRes::s_queries;
+unsigned int SyncRes::s_outqueries;
+bool SyncRes::s_log;
 
 #define LOG if(s_log)L<<Logger::Warning
 
 /** everything begins here - this is the entry point just after receiving a packet */
-template<>int SyncRes<>::beginResolve(const string &qname, const QType &qtype, vector<DNSResourceRecord>&ret)
+int SyncRes::beginResolve(const string &qname, const QType &qtype, vector<DNSResourceRecord>&ret)
 {
   set<GetBestNSAnswer> beenthere;
   s_queries++;
@@ -48,7 +48,7 @@ template<>int SyncRes<>::beginResolve(const string &qname, const QType &qtype, v
   return res;
 }
 
-template<class MultiPlexor>int SyncRes<MultiPlexor>::doResolve(const string &qname, const QType &qtype, vector<DNSResourceRecord>&ret, int depth, set<GetBestNSAnswer>& beenthere)
+int SyncRes::doResolve(const string &qname, const QType &qtype, vector<DNSResourceRecord>&ret, int depth, set<GetBestNSAnswer>& beenthere)
 {
   string prefix(d_prefix);
   prefix.append(depth, ' ');
@@ -78,7 +78,7 @@ template<class MultiPlexor>int SyncRes<MultiPlexor>::doResolve(const string &qna
   return res<0 ? RCode::ServFail : res;
 }
 
-template<class MultiPlexor>string SyncRes<MultiPlexor>::getA(const string &qname, int depth, set<GetBestNSAnswer>& beenthere)
+string SyncRes::getA(const string &qname, int depth, set<GetBestNSAnswer>& beenthere)
 {
   vector<DNSResourceRecord> res;
   string ret;
@@ -89,7 +89,7 @@ template<class MultiPlexor>string SyncRes<MultiPlexor>::getA(const string &qname
   return ret;
 }
 
-template<class MultiPlexor>void SyncRes<MultiPlexor>::getBestNSFromCache(const string &qname, set<DNSResourceRecord>&bestns, int depth, set<GetBestNSAnswer>& beenthere)
+void SyncRes::getBestNSFromCache(const string &qname, set<DNSResourceRecord>&bestns, int depth, set<GetBestNSAnswer>& beenthere)
 {
   string prefix(d_prefix), subdomain(qname);
   prefix.append(depth, ' ');
@@ -120,7 +120,7 @@ template<class MultiPlexor>void SyncRes<MultiPlexor>::getBestNSFromCache(const s
 	answer.qname=toLower(qname); answer.bestns=bestns;
 	if(beenthere.count(answer)) {
 	  LOG<<prefix<<qname<<": We have NS in cache for '"<<subdomain<<"' but part of LOOP! Trying less specific NS"<<endl;
-	  for(typename set<GetBestNSAnswer>::const_iterator j=beenthere.begin();j!=beenthere.end();++j)
+	  for( set<GetBestNSAnswer>::const_iterator j=beenthere.begin();j!=beenthere.end();++j)
 	    LOG<<prefix<<qname<<": beenthere: "<<j->qname<<" ("<<j->bestns.size()<<")"<<endl;
 	  bestns.clear();
 	}
@@ -136,7 +136,7 @@ template<class MultiPlexor>void SyncRes<MultiPlexor>::getBestNSFromCache(const s
 
 
 /** doesn't actually do the work, leaves that to getBestNSFromCache */
-template<class MultiPlexor>string SyncRes<MultiPlexor>::getBestNSNamesFromCache(const string &qname,set<string>& nsset, int depth, set<GetBestNSAnswer>&beenthere)
+string SyncRes::getBestNSNamesFromCache(const string &qname,set<string>& nsset, int depth, set<GetBestNSAnswer>&beenthere)
 {
   string subdomain(qname);
 
@@ -150,7 +150,7 @@ template<class MultiPlexor>string SyncRes<MultiPlexor>::getBestNSNamesFromCache(
   return subdomain;
 }
 
-template<class MultiPlexor>bool SyncRes<MultiPlexor>::doCNAMECacheCheck(const string &qname, const QType &qtype, vector<DNSResourceRecord>&ret, int depth, int &res)
+bool SyncRes::doCNAMECacheCheck(const string &qname, const QType &qtype, vector<DNSResourceRecord>&ret, int depth, int &res)
 {
   string prefix(d_prefix), tuple=toLower(qname)+"|CNAME";
   prefix.append(depth, ' ');
@@ -182,7 +182,7 @@ template<class MultiPlexor>bool SyncRes<MultiPlexor>::doCNAMECacheCheck(const st
   return false;
 }
 
-template<class MultiPlexor>bool SyncRes<MultiPlexor>::doCacheCheck(const string &qname, const QType &qtype, vector<DNSResourceRecord>&ret, int depth, int &res)
+bool SyncRes::doCacheCheck(const string &qname, const QType &qtype, vector<DNSResourceRecord>&ret, int depth, int &res)
 {
   string prefix(d_prefix), tuple;
   prefix.append(depth, ' ');
@@ -232,7 +232,7 @@ template<class MultiPlexor>bool SyncRes<MultiPlexor>::doCacheCheck(const string
   return false;
 }
 
-template<class MultiPlexor>bool SyncRes<MultiPlexor>::moreSpecificThan(const string& a, const string &b)
+bool SyncRes::moreSpecificThan(const string& a, const string &b)
 {
   int counta=!a.empty(), countb=!b.empty();
   
@@ -245,7 +245,7 @@ template<class MultiPlexor>bool SyncRes<MultiPlexor>::moreSpecificThan(const str
   return counta>countb;
 }
 
-template<class MultiPlexor>vector<string> SyncRes<MultiPlexor>::shuffle(set<string> &nameservers)
+vector<string> SyncRes::shuffle(set<string> &nameservers)
 {
   vector<string> rnameservers;
   for(set<string>::const_iterator i=nameservers.begin();i!=nameservers.end();++i)
@@ -256,7 +256,7 @@ template<class MultiPlexor>vector<string> SyncRes<MultiPlexor>::shuffle(set<stri
 }
 
 /** returns -1 in case of no results, rcode otherwise */
-template<class MultiPlexor>int SyncRes<MultiPlexor>::doResolveAt(set<string> nameservers, string auth, const string &qname, const QType &qtype, vector<DNSResourceRecord>&ret, 
+int SyncRes::doResolveAt(set<string> nameservers, string auth, const string &qname, const QType &qtype, vector<DNSResourceRecord>&ret, 
 		int depth, set<GetBestNSAnswer>&beenthere)
 {
   string prefix(d_prefix);
@@ -355,7 +355,7 @@ template<class MultiPlexor>int SyncRes<MultiPlexor>::doResolveAt(set<string> nam
 	  newtarget=i->content;
 	}
 	// for ANY answers we *must* have an authoritive answer
-	else if(i->d_place==DNSResourceRecord::ANSWER && i->qname==qname && (i->qtype==qtype || ( qtype==QType(QType::ANY) && aabit)))  {
+	else if(i->d_place==DNSResourceRecord::ANSWER && toLower(i->qname)==toLower(qname) && (i->qtype==qtype || ( qtype==QType(QType::ANY) && aabit)))  {
 	  LOG<<prefix<<qname<<": answer is in: resolved to '"<<i->content<<"|"<<i->qtype.getName()<<"'"<<endl;
 	  done=true;
 	  ret.push_back(*i);
@@ -403,7 +403,7 @@ template<class MultiPlexor>int SyncRes<MultiPlexor>::doResolveAt(set<string> nam
   return -1;
 }
 
-template<class MultiPlexor>void SyncRes<MultiPlexor>::addCruft(const string &qname, vector<DNSResourceRecord>& ret)
+void SyncRes::addCruft(const string &qname, vector<DNSResourceRecord>& ret)
 {
   for(vector<DNSResourceRecord>::const_iterator k=ret.begin();k!=ret.end();++k)  // don't add stuff to an NXDOMAIN!
     if(k->d_place==DNSResourceRecord::AUTHORITY && k->qtype==QType(QType::SOA))
@@ -433,7 +433,7 @@ template<class MultiPlexor>void SyncRes<MultiPlexor>::addCruft(const string &qna
   LOG<<d_prefix<<qname<<": Done with additional processing"<<endl;
 }
 
-template<class MultiPlexor>void SyncRes<MultiPlexor>::addAuthorityRecords(const string& qname, vector<DNSResourceRecord>& ret, int depth)
+void SyncRes::addAuthorityRecords(const string& qname, vector<DNSResourceRecord>& ret, int depth)
 {
   set<DNSResourceRecord> bestns;
   set<GetBestNSAnswer>beenthere;
diff --git a/pdns/syncres.hh b/pdns/syncres.hh
index 7b31525a5..401d24f72 100644
--- a/pdns/syncres.hh
+++ b/pdns/syncres.hh
@@ -14,7 +14,7 @@ void replaceCache(const string &qname, const QType &qt, const set<DNSResourceRec
 int getCache(const string &qname, const QType& qt, set<DNSResourceRecord>* res=0);
 
 
-template<class MultiPlexor=LWRes> class SyncRes
+class SyncRes
 {
 public:
   SyncRes() : d_outqueries(0), d_cacheonly(false), d_nocache(false){}
@@ -60,7 +60,7 @@ private:
   static bool s_log;
   bool d_cacheonly;
   bool d_nocache;
-  MultiPlexor d_lwr;
+  LWRes d_lwr;
   static map<string,string> s_negcache;
   struct GetBestNSAnswer
   {