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 2018/12/09 18:43:30 UTC

[Bug 7667] New: division by zero possible in latest update

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

            Bug ID: 7667
           Summary: division by zero possible in latest update
           Product: Spamassassin
           Version: 3.3.2
          Hardware: PC
                OS: Linux
            Status: NEW
          Severity: normal
          Priority: P2
         Component: Rules
          Assignee: dev@spamassassin.apache.org
          Reporter: nerijus@users.sourceforge.net
  Target Milestone: Undefined

CentOS 5, the latest update cannot be installed:
/etc/cron.daily/sa_update:

config: invalid expression for rule T_MIXED_ES: "( __LOWER_E > 20 ) && (
__E_LIKE_LETTER > ( (__LOWER_E * 14 ) / 10) ) && ( ( __E_LIKE_LETTER /
__LOWER_E ) < 10 )": division by zero possible

I assume it is because of __E_LIKE_LETTER / __LOWER_E. Is it possible to
rewrite it, or is the perl version too old in CentOS 5?

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

[Bug 7667] division by zero possible in latest update

Posted by bu...@bugzilla.spamassassin.org.
https://bz.apache.org/SpamAssassin/show_bug.cgi?id=7667

--- Comment #5 from Bill Cole <bi...@apache.org> ---
(In reply to RW from comment #4)
> Why not just rewrite it as
> 
>  ... && (__E_LIKE_LETTER  <  10*__LOWER_E )
> 
> and have done with it.

Good idea.

Done.

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

[Bug 7667] division by zero possible in latest update

Posted by bu...@bugzilla.spamassassin.org.
https://bz.apache.org/SpamAssassin/show_bug.cgi?id=7667

nerijus@users.sourceforge.net changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |nerijus@users.sourceforge.n
                   |                            |et

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

[Bug 7667] division by zero possible in latest update

Posted by bu...@bugzilla.spamassassin.org.
https://bz.apache.org/SpamAssassin/show_bug.cgi?id=7667

RW <rw...@googlemail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |rwmaillists@googlemail.com

--- Comment #4 from RW <rw...@googlemail.com> ---
Why not just rewrite it as

 ... && (__E_LIKE_LETTER  <  10*__LOWER_E )

and have done with it.

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

[Bug 7667] division by zero possible in latest update

Posted by bu...@bugzilla.spamassassin.org.
https://bz.apache.org/SpamAssassin/show_bug.cgi?id=7667

nerijus@users.sourceforge.net changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         Resolution|WORKSFORME                  |FIXED

--- Comment #3 from nerijus@users.sourceforge.net ---
Version 3.3.2 (spamassassin-3.3.2-56.el5). But the interesting thing is, that
on CentOS 6 I have spamassassin-3.3.1-3.el6, and update worked there, there is
T_MIXED_ES rule in /var/lib/spamassassin/3.003001/72_active.cf.
I will try to build 3.4.2.

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

[Bug 7667] division by zero possible in latest update

Posted by bu...@bugzilla.spamassassin.org.
https://bz.apache.org/SpamAssassin/show_bug.cgi?id=7667

--- Comment #6 from nerijus@users.sourceforge.net ---
I built and installed 3.4.2, but still get the same error on CentOS 5:

# sa-update 
config: invalid expression for rule T_MIXED_ES: "( __LOWER_E > 20 ) && (
__E_LIKE_LETTER > ( (__LOWER_E * 14 ) / 10) ) && ( ( __E_LIKE_LETTER /
__LOWER_E ) < 10 )": division by zero possible

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

[Bug 7667] division by zero possible in latest update

Posted by bu...@bugzilla.spamassassin.org.
https://bz.apache.org/SpamAssassin/show_bug.cgi?id=7667

Bill Cole <bi...@apache.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |billcole@apache.org
             Status|NEW                         |RESOLVED
         Resolution|---                         |WORKSFORME

--- Comment #1 from Bill Cole <bi...@apache.org> ---
We don't have a deep analysis of this bug, but it has only been reported with
obsolete unsupported versions of SpamAssassin (3.3.x) which would be consistent
with CentOS 5.x or any other EL5-family distribution. The specific error
message is generated by SpamAssassin, not Perl itself. The relevant code (in
Mail::SpamAssassin::Conf::Parser) has undergone extensive revision between
3.3.2 and 3.4.2..

You may be able to upgrade SA to v3.4.2 using CPAN, because in principle SA
v3.4.2 supports Perl v5.8.1 or newer. If you do so and still encounter this
issue, please reopen this bug.

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

[Bug 7667] division by zero possible in latest update

Posted by bu...@bugzilla.spamassassin.org.
https://bz.apache.org/SpamAssassin/show_bug.cgi?id=7667

Kevin A. McGrail <km...@apache.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |kmcgrail@apache.org

--- Comment #2 from Kevin A. McGrail <km...@apache.org> ---
What version of spamassassin is on the system? We can at least add a version
conditional to the rule.

Regards,
KAM

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