4.23-11: major bug: virus warning text itself appears in quarantine

[Peter Bonivart]

Since I don't catch them, or rather quarantine them, I guess I have to
generate such a message myself. I try that tomorrow when I'm back at work.

If it has anything to do with it I applied the following two patches
from Mariano that you approved of.

 >--- Message.pm.orig     Wed Sep 10 10:27:53 2003
 >+++ Message.pm  Wed Sep 10 10:29:45 2003
 >@@ -1923,7 +1923,7 @@
 >        !$this->{nameinfected} && # They want the tagging & not
already tagged
 >        !$global::MS->{mta}->TextStartsHeader($this, 'Subject:',
 > $contenttag))
 >{
 >      if (MailScanner::Config::Value('contentprependsubject',$this)) {
 >-      $global::MS->{mta}->PrependHeader($this, 'Subject:', $nametag,
' ');
 >+      $global::MS->{mta}->PrependHeader($this, 'Subject:', $contenttag, '
 >');
 >        $subjectchanged = 1;
 >      }
 >    } else {

 >--- Message.pm.orig     Wed Sep 10 10:27:53 2003
 >+++ Message.pm  Fri Sep 12 15:57:35 2003
 >@@ -995,6 +995,7 @@
 >    }
 >    while (($file, $text) = each %{$this->{namereports}}) {
 >      #print STDERR "Adding file $file report $text\n";
 >+    $text =~ s/\n(.)/\n$Name$1/g;
 >      $this->{allreports}{$file} .= $Name . $text;
 >    }
 >    while (($file, $text) = each %{$this->{nametypes}}) {

/Peter Bonivart

--Unix lovers do it in the Sun

Sun Fire V210, Solaris 9, MailScanner 4.23-11, SpamAssassin 2.55, ClamAV
  20030829

Julian Field wrote:
> I have just tried that combination as well. Works perfectly for me. Can you
> zip up and send me a sample message which you can prove breaks it please?