spamassassin ignores maxhits on tflags multiple

Bug #1193352 reported by ZEROSPAM
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
spamassassin (Ubuntu)
Invalid
Undecided
Unassigned

Bug Description

Using spamassassin 3.3.2 on precise with perl 5.14.2.

When trying spamassassin -D --lint 2>&1 < test.eml , we realized that some special rules had not the expected behavior.

One can use tflags on a rule, and specify multiple. This causes SA not to stop after the first hit but instead comtinue to evaluate the rule against the mail.

And one can also specify maxhits=N, with N the maximum number of occurrences he wants to checks for. This is usefull to count only a few occurences but not too much, to avoid loosing resources or simply count an exact number of matches on the rule.

But the maxhits is ignored on precise. This happens because the /usr/share/perl5/Mail/SpamAssassin/Plugin/Check.pm is not up to date (compared with ~/spamassassin-trunk/lib/Mail/SpamAssassin/Plugin/Check.pm directly from the svn checkout of SA project, done with svn checkout http://svn.apache.org/repos/asf/spamassassin/trunk spamassassin-trunk ).

This causes SA to ignore the maxhits instruction on tflags multiple rules. Ultimately this is a potential resource sink hole and is, in all cases, inefficient and deviate from the SA project itself.

We couldn't find any reason or explanation nowere, if there is one, please explain.

Patch is simple: update /usr/share/perl5/Mail/SpamAssassin/Plugin/Check.pm with the Check.pm coming from SA svn checkout.

Tested successfully on a precise server. Should work on any Ubuntu version.

Attached is the diff between those two files.

Revision history for this message
ZEROSPAM (noc-b) wrote :
Revision history for this message
ZEROSPAM (noc-b) wrote :

We realized that this issue is not an issue.

The maxhits feature was committed in SA after the release of 3.3.2.

It means that Ubuntu will have this feature with the next version of SA (3.4?)

Sorry for any inconvenience.

Changed in spamassassin (Ubuntu):
status: New → Invalid
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.