clamdscan plugin defaults
[email protected] (Matt Simerson) Mon, 29 Apr 2013 19:30:52 -0700
| Newsgroups | perl.qpsmtpd |
|---|---|
| Message-ID | <[email protected]> |
Within the register sub of the clamdscan plugin, is this little nugget:
# Set some sensible defaults
$self->{'_args'}{'deny_viruses'} ||=3D 'yes';
$self->{'_args'}{'max_size'} ||=3D 128;
$self->{'_args'}{'scan_all'} ||=3D 0;
Having a default enable for denying viruses is sensible enough.=20
But a max_size of 128K? You mean all a virus author needs to do is =
attach an image to his virus laden message to evade virus scanning on a =
qpsmtpd server? Is that really a sensible default? =20
My first inclination is that max_size should default to whatever =
$config->data_bytes is set to. Why would such a low limit be considered =
sensible?
The other thing I'm questioning is why scan_all=3D0 is the 'sensible' =
default. If one is going to bother running a virus scanner, it would =
seem the "safe" choice is to scan everything. Should it be as easy as =
inserting an illegal character into the Content-Type field value (which =
would get ignored later), to bypass multipart detection, and thus virus =
scanning?
Matt=