You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@spamassassin.apache.org by mm...@apache.org on 2013/01/29 15:48:17 UTC
svn commit: r1439916 -
/spamassassin/trunk/lib/Mail/SpamAssassin/DnsResolver.pm
Author: mmartinec
Date: Tue Jan 29 14:48:16 2013
New Revision: 1439916
URL: http://svn.apache.org/viewvc?rev=1439916&view=rev
Log:
added a DNS check for multiple question sections, just in case
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=1439916&r1=1439915&r2=1439916&view=diff
==============================================================================
--- spamassassin/trunk/lib/Mail/SpamAssassin/DnsResolver.pm (original)
+++ spamassassin/trunk/lib/Mail/SpamAssassin/DnsResolver.pm Tue Jan 29 14:48:16 2013
@@ -603,6 +603,9 @@ sub _packet_id {
my $id = $header->id;
my @questions = $packet->question;
+ @questions <= 1
+ or warn "dns: packet has multiple questions: " . $packet->string . "\n";
+
if ($questions[0]) {
# Bug 6232: Net::DNS::Packet::new is not consistent in keeping data in
# sections of a packet either as original bytes or presentation-encoded: