spamassassin-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From mmarti...@apache.org
Subject svn commit: r921096 - /spamassassin/trunk/lib/Mail/SpamAssassin/DnsResolver.pm
Date Tue, 09 Mar 2010 20:19:22 GMT
Author: mmartinec
Date: Tue Mar  9 20:19:22 2010
New Revision: 921096

URL: http://svn.apache.org/viewvc?rev=921096&view=rev
Log:
Bug 6362: remove a call to connect_sock_if_reqd() from
sub nameservers, it has no business there, overloading
its functionality

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

Modified: spamassassin/trunk/lib/Mail/SpamAssassin/DnsResolver.pm
URL: http://svn.apache.org/viewvc/spamassassin/trunk/lib/Mail/SpamAssassin/DnsResolver.pm?rev=921096&r1=921095&r2=921096&view=diff
==============================================================================
--- spamassassin/trunk/lib/Mail/SpamAssassin/DnsResolver.pm (original)
+++ spamassassin/trunk/lib/Mail/SpamAssassin/DnsResolver.pm Tue Mar  9 20:19:22 2010
@@ -181,7 +181,6 @@ sub nameservers {
     }
     $self->{available_dns_servers} = \@ns_addr_port;
   }
-  $self->connect_sock_if_reqd();
   return @{$self->{available_dns_servers}};
 }
 
@@ -204,7 +203,7 @@ sub connect_sock {
   my $errno;
 
   # list of name servers: [addr]:port entries
-  $self->nameservers  if !$self->{available_dns_servers};
+  $self->nameservers()  if !$self->{available_dns_servers};
   my @ns_addr_port = @{$self->{available_dns_servers}};
   # use the first name server in a list
   my($ns_addr,$ns_port); local($1,$2);



Mime
View raw message