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 2017/04/11 18:37:55 UTC

[Bug 7404] Bad regexp (and logic) in MS::PerMsgStatus::get_content_preview

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

Mark Martinec <Ma...@ijs.si> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
            Summary|MS::PerMsgStatus::get_conte |Bad regexp (and logic) in
                   |nt_preview                  |MS::PerMsgStatus::get_conte
                   |                            |nt_preview
   Target Milestone|Undefined                   |3.4.2

--- Comment #1 from Mark Martinec <Ma...@ijs.si> ---
(sorry for hitting a submit prematurely on an unfinished PR)

Testing with 5.25.11 (soon to be be released as 5.26) fills the log
with warnings on a suspicious regexp, as shown above - an x{,n}
is not interpreted as x{0,n}, and even if it were, the logic is wrong
in that code.

fixing in trunk:
  Sending lib/Mail/SpamAssassin/PerMsgStatus.pm
Committed revision 1791010.

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