You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@spamassassin.apache.org by gb...@apache.org on 2021/05/19 07:18:39 UTC
svn commit: r1890021 - in /spamassassin:
branches/3.4/lib/Mail/SpamAssassin/Plugin/TextCat.pm
trunk/lib/Mail/SpamAssassin/Plugin/TextCat.pm
Author: gbechis
Date: Wed May 19 07:18:39 2021
New Revision: 1890021
URL: http://svn.apache.org/viewvc?rev=1890021&view=rev
Log:
avoid a division by zero error
Modified:
spamassassin/branches/3.4/lib/Mail/SpamAssassin/Plugin/TextCat.pm
spamassassin/trunk/lib/Mail/SpamAssassin/Plugin/TextCat.pm
Modified: spamassassin/branches/3.4/lib/Mail/SpamAssassin/Plugin/TextCat.pm
URL: http://svn.apache.org/viewvc/spamassassin/branches/3.4/lib/Mail/SpamAssassin/Plugin/TextCat.pm?rev=1890021&r1=1890020&r2=1890021&view=diff
==============================================================================
--- spamassassin/branches/3.4/lib/Mail/SpamAssassin/Plugin/TextCat.pm (original)
+++ spamassassin/branches/3.4/lib/Mail/SpamAssassin/Plugin/TextCat.pm Wed May 19 07:18:39 2021
@@ -459,7 +459,11 @@ sub classify {
my @results_tag;
foreach (@results[0..19]) {
last unless defined $_;
- push @results_tag, sprintf "%s:%s(%.02f)", $_, $results{$_}, $results{$_} / $best;
+ if($best != 0) {
+ push @results_tag, sprintf "%s:%s(%.02f)", $_, $results{$_}, $results{$_} / $best;
+ } else {
+ push @results_tag, sprintf "%s:%s(unknown)", $_, $results{$_};
+ }
}
$opts->{permsgstatus}->set_tag('TEXTCATRESULTS', join(' ', @results_tag));
Modified: spamassassin/trunk/lib/Mail/SpamAssassin/Plugin/TextCat.pm
URL: http://svn.apache.org/viewvc/spamassassin/trunk/lib/Mail/SpamAssassin/Plugin/TextCat.pm?rev=1890021&r1=1890020&r2=1890021&view=diff
==============================================================================
--- spamassassin/trunk/lib/Mail/SpamAssassin/Plugin/TextCat.pm (original)
+++ spamassassin/trunk/lib/Mail/SpamAssassin/Plugin/TextCat.pm Wed May 19 07:18:39 2021
@@ -459,7 +459,11 @@ sub classify {
my @results_tag;
foreach (@results[0..19]) {
last unless defined $_;
- push @results_tag, sprintf "%s:%s(%.02f)", $_, $results{$_}, $results{$_} / $best;
+ if($best != 0) {
+ push @results_tag, sprintf "%s:%s(%.02f)", $_, $results{$_}, $results{$_} / $best;
+ } else {
+ push @results_tag, sprintf "%s:%s(unknown)", $_, $results{$_};
+ }
}
$opts->{permsgstatus}->set_tag('TEXTCATRESULTS', join(' ', @results_tag));