You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@spamassassin.apache.org by bu...@spamassassin.apache.org on 2021/03/07 21:51:51 UTC

[Bug 7875] AskDNS plugin does not correctly handle CNAMEs leading to TXTs

https://bz.apache.org/SpamAssassin/show_bug.cgi?id=7875

Henrik Krohns <ap...@hege.li> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         Resolution|---                         |FIXED
                 CC|                            |apache@hege.li
             Status|NEW                         |RESOLVED

--- Comment #3 from Henrik Krohns <ap...@hege.li> ---

The whole $qtype comparison completely unnecessary and breaks stuff, RR
comparison is done later here in if:

                 !grep($_ eq 'ANY' || $_ eq $rr_type, @$answer_types_ref) ) {
          # skip remaining tests on wrong RR type

.. which allows for ANY / multiple RR type queries to work too.

I spend some time looking through the code carefully, this should be safe to
remove for 3.4..

$pms->{askdns_map_dnskey_to_rules}{$dnskey}[$j++] = undef;

It fixes this bug and the multiple response bug discussed in Bug 7777.

Similar fixes committed to trunk also:

Sending        spamassassin-3.4/lib/Mail/SpamAssassin/Plugin/AskDNS.pm
Sending        trunk/lib/Mail/SpamAssassin/Plugin/AskDNS.pm
Transmitting file data ..done
Committing transaction...
Committed revision 1887305.

-- 
You are receiving this mail because:
You are the assignee for the bug.