avoid merging old and new additional data, fixes an issue caused by weird Akamai...

diff --git a/pdns/recursor_cache.cc b/pdns/recursor_cache.cc
index c7ac0a0de3805d522a4ee7347fb093e87ea299b4..14df3a56f371a9778c3722d630c65a19a4f6eb5e 100644 (file)
--- a/pdns/recursor_cache.cc
+++ b/pdns/recursor_cache.cc
@@ -242,6 +242,12 @@ void MemRecursorCache::replace(time_t now, const string &qname, const QType& qt,
   else
     ; //cerr<<"\tNot nuking"<<endl;
 
+  // make sure we don't accidentally merge old and new unauth data
+  if(!auth && !ce.d_auth) {
+    ce.d_records.clear();
+    isNew=true;
+  }
+
   // cerr<<"\tHave "<<content.size()<<" records to store\n";
   for(set<DNSResourceRecord>::const_iterator i=content.begin(); i != content.end(); ++i) {
     // cerr<<"To store: "<<i->content<<" with ttl/ttd "<<i->ttl<<endl;