You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@spamassassin.apache.org by mm...@apache.org on 2008/02/28 15:25:52 UTC

svn commit: r631982 - /spamassassin/trunk/lib/Mail/SpamAssassin/Plugin/Check.pm

Author: mmartinec
Date: Thu Feb 28 06:25:51 2008
New Revision: 631982

URL: http://svn.apache.org/viewvc?rev=631982&view=rev
Log:
A missing or failed eval rule function can mistakenly count as a rule hit, bug 5839

Modified:
    spamassassin/trunk/lib/Mail/SpamAssassin/Plugin/Check.pm

Modified: spamassassin/trunk/lib/Mail/SpamAssassin/Plugin/Check.pm
URL: http://svn.apache.org/viewvc/spamassassin/trunk/lib/Mail/SpamAssassin/Plugin/Check.pm?rev=631982&r1=631981&r2=631982&view=diff
==============================================================================
--- spamassassin/trunk/lib/Mail/SpamAssassin/Plugin/Check.pm (original)
+++ spamassassin/trunk/lib/Mail/SpamAssassin/Plugin/Check.pm Thu Feb 28 06:25:51 2008
@@ -968,6 +968,7 @@
       eval {
         $result = $self->' . $function . ' (@extraevalargs '. $argstr .' );  1;
       } or do {
+        $result = 0;
         $self->handle_eval_rule_errors($rulename);
       };