You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@spamassassin.apache.org by fe...@apache.org on 2005/02/24 19:36:54 UTC

svn commit: r155223 - spamassassin/trunk/lib/Mail/SpamAssassin/Dns.pm

Author: felicity
Date: Thu Feb 24 10:36:53 2005
New Revision: 155223

URL: http://svn.apache.org/viewcvs?view=rev&rev=155223
Log:
bug 4154: check for dns_available wasn't trying multiple times as it was supposed to.  let a 'softfail' (ie: Net::DNS didn't die) do a retry.

Modified:
    spamassassin/trunk/lib/Mail/SpamAssassin/Dns.pm

Modified: spamassassin/trunk/lib/Mail/SpamAssassin/Dns.pm
URL: http://svn.apache.org/viewcvs/spamassassin/trunk/lib/Mail/SpamAssassin/Dns.pm?view=diff&r1=155222&r2=155223
==============================================================================
--- spamassassin/trunk/lib/Mail/SpamAssassin/Dns.pm (original)
+++ spamassassin/trunk/lib/Mail/SpamAssassin/Dns.pm Thu Feb 24 10:36:53 2005
@@ -640,11 +640,15 @@
     my $domain = splice(@domains, rand(@domains), 1);
     dbg("dns: trying ($retry) $domain...");
     my $result = $self->lookup_ns($domain);
-    if(defined $result && scalar @$result > 0) {
-      if ( $result ) {
+    if(defined $result) {
+      if (scalar @$result > 0) {
         dbg("dns: NS lookup of $domain succeeded => DNS available (set dns_available to override)");
         $IS_DNS_AVAILABLE = 1;
         last;
+      }
+      else {
+        dbg("dns: NS lookup of $domain failed, no results found");
+	next;
       }
     }
     else {