svn commit: r410038 - in /spamassassin/trunk/lib/Mail/SpamAssassin: PerMsgStatus.pm Plugin/DCC.pm Plugin/Pyzor.pm

[do...@apache.org]

Author: dos
Date: Mon May 29 00:06:09 2006
New Revision: 410038

URL: http://svn.apache.org/viewvc?rev=410038&view=rev
Log:
bug 4924: initialize tag data for _DCCB_, _DCCR_, _PYZOR_, _RBL_, _BAYESTC*_

Modified:
    spamassassin/trunk/lib/Mail/SpamAssassin/PerMsgStatus.pm
    spamassassin/trunk/lib/Mail/SpamAssassin/Plugin/DCC.pm
    spamassassin/trunk/lib/Mail/SpamAssassin/Plugin/Pyzor.pm

Modified: spamassassin/trunk/lib/Mail/SpamAssassin/PerMsgStatus.pm
URL: http://svn.apache.org/viewvc/spamassassin/trunk/lib/Mail/SpamAssassin/PerMsgStatus.pm?rev=410038&r1=410037&r2=410038&view=diff
==============================================================================
--- spamassassin/trunk/lib/Mail/SpamAssassin/PerMsgStatus.pm (original)
+++ spamassassin/trunk/lib/Mail/SpamAssassin/PerMsgStatus.pm Mon May 29 00:06:09 2006
@@ -1290,7 +1290,8 @@
       $data = $data->(@_);
     }
   }
-  else {
+  # known valid tags that might not get defined in some circumstances
+  elsif ($tag !~ /^(?:BAYESTC(?:|LEARNED|SPAMMY|HAMMY)|RBL)$/) {
     return undef;
   }
   $data = "" unless defined $data;

Modified: spamassassin/trunk/lib/Mail/SpamAssassin/Plugin/DCC.pm
URL: http://svn.apache.org/viewvc/spamassassin/trunk/lib/Mail/SpamAssassin/Plugin/DCC.pm?rev=410038&r1=410037&r2=410038&view=diff
==============================================================================
--- spamassassin/trunk/lib/Mail/SpamAssassin/Plugin/DCC.pm (original)
+++ spamassassin/trunk/lib/Mail/SpamAssassin/Plugin/DCC.pm Mon May 29 00:06:09 2006
@@ -362,9 +362,15 @@
 
   # short-circuit if there's already a X-DCC header with value of
   # "bulk" from an upstream DCC check
-  if ($permsgstatus->get('ALL') =~ /^X-DCC-(?:[^:]{1,80}-)?Metrics:.*bulk/m) {
+  if ($permsgstatus->get('ALL') =~ /^X-DCC-([^:]{1,80})?-?Metrics:.*bulk/m) {
+    $permsgstatus->{tag_data}->{DCCB} = $1;
+    $permsgstatus->{tag_data}->{DCCR} = "bulk";
     return 1;
   }
+
+  # initialize valid tags
+  $permsgstatus->{tag_data}->{DCCB} = "";
+  $permsgstatus->{tag_data}->{DCCR} = "";
 
   $self->get_dcc_interface();
   return 0 if $self->{dcc_disabled};

Modified: spamassassin/trunk/lib/Mail/SpamAssassin/Plugin/Pyzor.pm
URL: http://svn.apache.org/viewvc/spamassassin/trunk/lib/Mail/SpamAssassin/Plugin/Pyzor.pm?rev=410038&r1=410037&r2=410038&view=diff
==============================================================================
--- spamassassin/trunk/lib/Mail/SpamAssassin/Plugin/Pyzor.pm (original)
+++ spamassassin/trunk/lib/Mail/SpamAssassin/Plugin/Pyzor.pm Mon May 29 00:06:09 2006
@@ -215,6 +215,9 @@
 sub check_pyzor {
   my ($self, $permsgstatus, $full) = @_;
 
+  # initialize valid tags
+  $permsgstatus->{tag_data}->{PYZOR} = "";
+
   $self->get_pyzor_interface();
   return 0 unless $self->{pyzor_available};