[Bug 6871] New: FreeMail plugin not checking Reply-To properly

[bu...@bugzilla.spamassassin.org]

https://issues.apache.org/SpamAssassin/show_bug.cgi?id=6871

            Bug ID: 6871
           Summary: FreeMail plugin not checking Reply-To properly
           Product: Spamassassin
           Version: SVN Trunk (Latest Devel Version)
          Hardware: All
                OS: All
            Status: NEW
          Severity: major
          Priority: P2
         Component: Plugins
          Assignee: dev@spamassassin.apache.org
          Reporter: hege@hege.li
    Classification: Unclassified

Long standing rookie mistake with index(). Recommended for everyone to upgrade
the plugin. :-)

Any Reply-To header (or similar) with "" or <> stanza would not be detected
(except when the domain to match was a wildcard one like yahoo.*).

--- lib/Mail/SpamAssassin/Plugin/FreeMail.pm    (revision 1416456)
+++ lib/Mail/SpamAssassin/Plugin/FreeMail.pm    (working copy)
@@ -425,7 +425,7 @@
         }
     }

-    my $email = lc($pms->get(index($header,':') ? $header : $header.":addr"));
+    my $email = lc($pms->get(index($header,':') >= 0 ? $header :
$header.":addr"));

     if ($email eq '') {
         dbg("header $header not found from mail");

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

[Bug 6871] FreeMail plugin not checking Reply-To properly

[bu...@bugzilla.spamassassin.org]

https://issues.apache.org/SpamAssassin/show_bug.cgi?id=6871

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

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

--- Comment #1 from Henrik Krohns <he...@hege.li> ---

It also affects From: similarly. But atleast FREEMAIL_FROM works because it
calls all_from_addrs() internally.

Sending        lib/Mail/SpamAssassin/Plugin/FreeMail.pm
Transmitting file data .
Committed revision 1416457.

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