From: bert hubert <bert.hubert@netherlabs.nl>
Date: Wed, 2 Mar 2016 14:02:12 +0000 (+0100)
Subject: reinstate old behaviour that we do process authoritative responses that are header... 
X-Git-Tag: rec-4.0.0-alpha2~24^2
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=refs%2Fpull%2F3497%2Fhead;p=thirdparty%2Fpdns.git

reinstate old behaviour that we do process authoritative responses that are header only to hasten the timeout
---

diff --git a/pdns/pdns_recursor.cc b/pdns/pdns_recursor.cc
index 3042f8e5de..3d56ab172d 100644
--- a/pdns/pdns_recursor.cc
+++ b/pdns/pdns_recursor.cc
@@ -1949,11 +1949,12 @@ void handleUDPServerResponse(int fd, FDMultiplexer::funcparam_t& var)
   }
   else {
     try {
-      pident.domain=DNSName(data, len, 12, false, &pident.type); // don't copy this from above - we need to do the actual read
+      if(len > 12)
+        pident.domain=DNSName(data, len, 12, false, &pident.type); // don't copy this from above - we need to do the actual read
     }
     catch(std::exception& e) {
       g_stats.serverParseError++; // won't be fed to lwres.cc, so we have to increment
-      L<<Logger::Warning<<"Error in packet from "<< fromaddr.toStringWithPort() << ": "<<e.what() << endl;
+      L<<Logger::Warning<<"Error in packet from remote nameserver "<< fromaddr.toStringWithPort() << ": "<<e.what() << endl;
       return;
     }
   }