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 2005/02/19 07:24:39 UTC

[Bug 3650] [patch] Can't call SpamAssassin->finish(), multiple finish() errors

http://bugzilla.spamassassin.org/show_bug.cgi?id=3650





------- Additional Comments From peter@unlikejam.dreamhost.com  2005-02-18 22:24 -------
Related to better handling of finish() calls, calling finish twice on a
PerMsgStatus object fails fatally when call_plugins can't be found.

Suggested patch for PerMsgStatus.pm to go with the others already proposed:

--- ./PerMsgStatus.pm.orig  Tue Feb  8 10:45:55 2005
+++ ./PerMsgStatus.pm   Sat Feb 19 15:51:45 2005
@@ -1224,7 +1224,7 @@
 
   $self->{main}->call_plugins ("per_msg_finish", {
          permsgstatus => $self
-       });
+       }) if exists $self->{main};
 
   foreach(keys %{$self}) {
     delete $self->{$_};




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