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...@bugzilla.spamassassin.org on 2013/08/15 15:02:42 UTC
[Bug 6968] New: Bleadperl v5.19.2-257-gc30fc27 fixes a bug in perl
which Mail/SpamAssassin/Plugin/FreeMail.pm relies
https://issues.apache.org/SpamAssassin/show_bug.cgi?id=6968
Bug ID: 6968
Summary: Bleadperl v5.19.2-257-gc30fc27 fixes a bug in perl
which Mail/SpamAssassin/Plugin/FreeMail.pm relies
Product: Spamassassin
Version: 3.3.2
Hardware: PC
OS: Linux
Status: NEW
Severity: normal
Priority: P2
Component: Plugins
Assignee: dev@spamassassin.apache.org
Reporter: andreas.koenig.7os6VVqR@franz.ak.mind.de
As per subject. The relevant ticket where this fact was discovered is
https://rt.perl.org/rt3//Public/Bug/Display.html?id=119125
The patch to fix the problem for both old and new perls is:
--- lib/Mail/SpamAssassin/Plugin/FreeMail.pm~ 2011-06-07 01:59:17.000000000
+0200
+++ lib/Mail/SpamAssassin/Plugin/FreeMail.pm 2013-08-15 14:46:28.236376372
+0200
@@ -120,10 +120,10 @@
# full email regex
my $email_regex = qr/
(?=.{0,64}\@) # limit userpart to 64 chars
(and speed up searching?)
- (?<![a-z0-9!#$%&'*+\/=?^_`{|}~-]) # start boundary
+ (?<![a-z0-9!#\$%&'*+\/=?^_`{|}~-]) # start boundary
( # capture email
- [a-z0-9!#$%&'*+\/=?^_`{|}~-]+ # no dot in beginning
- (?:\.[a-z0-9!#$%&'*+\/=?^_`{|}~-]+)* # no consecutive dots, no ending dot
+ [a-z0-9!#\$%&'*+\/=?^_`{|}~-]+ # no dot in beginning
+ (?:\.[a-z0-9!#\$%&'*+\/=?^_`{|}~-]+)* # no consecutive dots, no
ending dot
\@
(?:[a-z0-9](?:[a-z0-9-]{0,59}[a-z0-9])?\.){1,4} # max 4x61 char parts
(should be enough?)
${tlds} # ends with valid tld
Note that the change has no effect on current tests.
HTH && Regards,
--
You are receiving this mail because:
You are the assignee for the bug.
[Bug 6968] Bleadperl v5.19.2-257-gc30fc27 fixes a bug in perl which
Mail/SpamAssassin/Plugin/FreeMail.pm relies
Posted by bu...@bugzilla.spamassassin.org.
https://issues.apache.org/SpamAssassin/show_bug.cgi?id=6968
Mark Martinec <Ma...@ijs.si> changed:
What |Removed |Added
----------------------------------------------------------------------------
Target Milestone|Undefined |3.4.0
--- Comment #1 from Mark Martinec <Ma...@ijs.si> ---
Great catch, thanks for letting us know!
The fix will be in 3.4.0.
trunk:
Bug 6968: quote a $ within a character class
(Bleadperl v5.19.2-257-gc30fc27 fixes a bug in perl
which Mail/SpamAssassin/Plugin/FreeMail.pm relies)
Sending lib/Mail/SpamAssassin/Plugin/FreeMail.pm
Committed revision 1514524.
--
You are receiving this mail because:
You are the assignee for the bug.
[Bug 6968] Bleadperl v5.19.2-257-gc30fc27 fixes a bug in perl which
Mail/SpamAssassin/Plugin/FreeMail.pm relies
Posted by bu...@bugzilla.spamassassin.org.
https://issues.apache.org/SpamAssassin/show_bug.cgi?id=6968
Karsten Bräckelmann <gu...@rudersport.de> changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|NEW |RESOLVED
Resolution|--- |FIXED
--- Comment #2 from Karsten Bräckelmann <gu...@rudersport.de> ---
(In reply to Mark Martinec from comment #1)
> Great catch, thanks for letting us know!
> The fix will be in 3.4.0.
> Committed revision 1514524.
That renders it RESOLVED FIXED.
--
You are receiving this mail because:
You are the assignee for the bug.