On Thu, 25 Mar 2004, Robert Covell wrote:

> sub filter_begin () {
>     if (stream_by_domain()) {
>         return;
>     }

>     if ((lc $Domain) eq "domain1.com") {
>         my($code, $category, $action) = message_contains_virus();
>         $FoundVirus = ($category eq "virus");

... etc ...

>     } else {
>         $action = "";
>     }

>     if ($FoundVirus) {

Danger, danger, danger!

You're never initializing $FoundVirus to false.

Change the top of your filter to read:

sub filter_begin () {
    if (stream_by_domain()) {
        return;
    }

    $FoundVirus = 0;

    if ((lc $Domain) eq "domain1.com") {
        my($code, $category, $action) = message_contains_virus();
        $FoundVirus = ($category eq "virus");

... etc ...

How much do you charge for virus-scanning?  I'll take 10% for this
fix, thanks! :-)

--
David.

_______________________________________________
Visit http://www.mimedefang.org and http://www.canit.ca
MIMEDefang mailing list
[EMAIL PROTECTED]
http://lists.roaringpenguin.com/mailman/listinfo/mimedefang

Reply via email to