bogus qf-file generated by mailscanner

[Piper Andreas]

Hello,

using mailscanner-3.20-6 with SA-2.30 on AIX 4.3 with sendmail-8.11
and Perl-5.005, I observe, that sometimes queue-Files in the delivering
queue are renamed from qf.. to Qf.. and the corresponding message is
not delivered. This happens for about 2 percent of messages marked
as spam.

Reason for this behaviour is, as far as I can see, that by generating the
output-qf-file mailscanner seems to split the header-line for the SA-result
in such a way, that a single line, containing only ')', without a leading
TAB-char, is generated. The ')' should correctly be at the end of the
previous line, to end the SA-report, standing alone on a single line it
is not recognized by sendmail and thus the qf-file is not accepted.

In the logs I found that for these messages the SA-report had an
additional whitespace at its end (before the ')'), which seems to cause
this behaviour. I included this patch into sendmail.pl (after line 285):

      $SASaysSpam = 0 unless $SAreport; # Solve bug with empty SAreports
+      $SAreport =~ s/\s+$// if $SAreport;

to get rid of the whitespace, and it is now working correctly for half a day.

Andreas Piper

________________________________________________________________________
Dr. Andreas Piper, Hochschulrechenzentrum der Philipps-Univ. Marburg
          Hans-Meerwein-Strasse, 35032 Marburg, Germany
Phone: +49 6421 28-23521  Fax: -26994  Email: piper at HRZ.Uni-Marburg.DE