From: bert hubert <bert.hubert@netherlabs.nl>
Date: Tue, 9 Dec 2014 14:00:52 +0000 (+0100)
Subject: improve cache hitrate
X-Git-Tag: rec-3.7.0-rc1~130
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=90974597aadaf1096e3fd0dc450be7422ea591a5;p=thirdparty%2Fpdns.git

improve cache hitrate
---

diff --git a/pdns/dns.cc b/pdns/dns.cc
index 42a3cafcd6..1678f7072c 100644
--- a/pdns/dns.cc
+++ b/pdns/dns.cc
@@ -101,7 +101,7 @@ bool dnspacketLessThan(const std::string& a, const std::string& b)
     int result=0;
     unsigned int n;
     for(n = 0; n < aLabelLen && n < bLabelLen; ++n) 
-      if((result = aSafe[aPos + n] - bSafe[bPos +n]))   // XXX this should perhaps be dns_tolower
+      if((result = dns_tolower(aSafe[aPos + n]) - dns_tolower(bSafe[bPos +n])))
         break;
     // cerr<<"Done loop, result="<<result<<", n = "<<n<<", aLabelLen="<<aLabelLen<<", bLabelLen="<<bLabelLen<<endl;
     if(result < 0)
diff --git a/pdns/recpacketcache.cc b/pdns/recpacketcache.cc
index d5c3ac4126..c9e89d12b1 100644
--- a/pdns/recpacketcache.cc
+++ b/pdns/recpacketcache.cc
@@ -62,6 +62,17 @@ bool RecursorPacketCache::getResponsePacket(const std::string& queryPacket, time
     memcpy(&id, queryPacket.c_str(), 2); 
     *responsePacket = iter->d_packet;
     responsePacket->replace(0, 2, (char*)&id, 2);
+    
+    string::size_type i=sizeof(dnsheader);
+
+    for(;;) {
+      int labellen = (unsigned char)queryPacket[i];
+      if(!labellen || i + labellen > responsePacket->size()) break;
+      i++;
+      responsePacket->replace(i, labellen, queryPacket, i, labellen);
+      i = i + labellen;
+    }
+
     d_hits++;
     moveCacheItemToBack(d_packetCache, iter);