DNS: avoid overflow in cache TTL for negative entries. Bug 1395
authorJeremy Harris <jgh146exb@wizmail.org>
Sun, 20 Sep 2015 20:47:10 +0000 (21:47 +0100)
committerJeremy Harris <jgh146exb@wizmail.org>
Sun, 20 Sep 2015 20:47:10 +0000 (21:47 +0100)
src/src/verify.c

index cb88f28..435570b 100644 (file)
@@ -3600,7 +3600,7 @@ cache the result in permanent memory. */
 
 else
   {
 
 else
   {
-  uint ttl = UINT_MAX;
+  uint ttl = 3600;
 
   store_pool = POOL_PERM;
 
 
   store_pool = POOL_PERM;
 
@@ -3633,7 +3633,10 @@ else
 
   Quite apart from one A6 RR generating multiple addresses, there are DNS
   lists that return more than one A record, so we must handle multiple
 
   Quite apart from one A6 RR generating multiple addresses, there are DNS
   lists that return more than one A record, so we must handle multiple
-  addresses generated in that way as well. */
+  addresses generated in that way as well.
+
+  Mark the cache entry with the "now" plus the minimum of the address TTLs,
+  or some suitably far-future time if none were found. */
 
   if (cb->rc == DNS_SUCCEED)
     {
 
   if (cb->rc == DNS_SUCCEED)
     {