Bug & Patch in qmail.pm parser of Mailscanner lib,

[Jérome MOLLIER PIERRET]

Hello again,

Finally, the regexp s/(u[^@]*F.*?\0)//  should be s/(u.*?F.*?\0)//
(instead of s/(u.*F.*?\0)// ).
Karthikeyan, S. will include it in the upcoming 5.0.1.6 OpenProtect
version.

Regards,

Jerome MOLLIER-PIERRET
Actinux (http://www.actinux.com)


Mollier Jerome wrote:

>Hello,
>
>First of all, we love the conception of Mailscanner and the clean & flexible
>"implementations" capabilities.
>
>We founded a bug in the ReadQf $MAILSCANNER_PATH/lib/MailScanner/Qmail.pm
>(line 210).
>
>If the recipient of an email is containing a "F" capital letter, then an
>incomplete regular expression isn't doing really his job (as the F is a
>separator in a file located in qmail queue.)
>
>Going deeper in ReadQf sub :
>-----BEGINING-----
>    if($to =~ /T/) {
>        $to =~ s/(u.*F.*?\0)//; #*HERE is the bug
>        do {
>            if($to =~ s/^T((.*?)\0)//) {
>                $TOFound = 1;
>                push @{$message->{to}}, $2;
>            }
>-----END-----
>
>replace the s/(u.*F.*?\0)// in order not to match the "F" in our recipient
>to the folling regexp s/(u[^@]*F.*?\0)// ... and that's all folks ...
>
>Best regards,
>
>Jerome MOLLIER-PIERRET
>Actinux (http://www.actinux.com)
>
>