clamdscan instead of clamscan

Joe, from the 'rpm -qi clamav-server':
....
This package contains files which are needed to execute the clamd-daemon. This daemon does not provide a system-wide service. Instead of, an instance of this daemon should be started for each service requiring it.

See the README file how this can be done with a minimum of effort.

So the README says:
To create individual clamd-instance take the following files and
modify/copy them in the suggested way:

clamd.conf:
* set LogFile, PidFile, LocalSocket and User to suitable values
* place this file into /etc/clamd.d with an unique service-name;
e.g. as /etc/clamd.d/<SERVICE>.conf

To make logfile rotation work properly, the LogFile should be
writable for the assigned User. Recommended way to reach this, is
to:
* make it owned by the User's *group*
* assign at least 0620 (u+rw,g+w) permissions

A suitable command might be
| # touch[logfile>
| # chgrp[user>[logfile>
| # chmod 0620 [logfile>

NEVER use 'clamav' as the user since he can modify the database.
This is the user who is running the application; e.g. for mimedefang
(http://www.roaringpenguin.com/mimedefang), the user might be
'defang'.Theoretically, distinct users could be used, but it must be
made sure that the application-user can write into the socket-file,
and that the clamd-user can access the files asked by the
application to be checked.

clamd.logrotate:
* set the correct value for the logfile
* place it into /etc/logrotate.d

clamd.sysconfig:
* set the name of the config-file and the local socket
* copy it to /etc/sysconfig/clamd.<SERVICE>

clamd.init:
* set the service-name
* place it into /etc/init.d/ with an unique name and activate it
(e.g. with /sbin/chkconfig clamd.<SERVICE> on)

Additionally, a symlink must be set to clamd in a way like
| # ln -s clamd /usr/sbin/clamd.<SERVICE>
and the directory for the socket file must be created (see 'LocalSocket'
in clamd.conf)
| # mkdir -p /var/run/clamd.<SERVICE>

This directory must be writable by the 'User' chosen in the config-file.

[[Disclaimer:
this file and the script/configfiles are not part of the official
clamav package.

Please send complaints and comments to
mailto:enrico.scholz@informatik.tu-chemnitz.de!]]

.....
Which is exactly what my instructions do above -- they create a specific instance of clamav for a single purpose use instead of creating a system-wide clamav.

Is it me, or did I not follow the README and I thought I RTFM and understood the Fine-FM.

What I'm really looking for, which it appears I may have to do it myself, is to extend the interface to allow for an extra field to append onto the command a configuration value. The current interface only allows for the executable without any additional configuration switches.

Would it be best that I send you the diffs for adding another field so that configuration switches can be added?

In that way, I won't have to mod the source after every update to Virtualmin Pro.

Joe,

In /usr/libexec/webmin/virtual-server/virtual-server-lib.pl,
there is a test for the "clamscan_cmd" variable:

if ($config{'virus'}) {
# Make sure ClamAV is installed and working
&has_command($config{'clamscan_cmd'}) ||
return &amp;text('index_evirus', &quot;<i>$config{'clamscan_cmd'}</i>&quot;, $clink);
local $out = `$config{'clamscan_cmd'} -[/dev/null 2&gt;&amp;1`;
$? &amp;&amp; return &amp;text('index_evirusrun', &quot;<i>$config{'clamscan_cmd'}</i>&quot;, &quot;[pre]$out[/pre]&quot;, $clink);
&amp;$second_print($text{'check_virusok'});
}

Instead of sending 'clamscan_cmd' to &amp;has_command, couldn't it be split and only the first component of clamscan_cmd be sent to &amp;has_command?

like...

split(' ',$config{'clamscan_cmd'},2)
&amp;has_command($_[[0]]);

if ($config{'virus'}) {
# Make sure ClamAV is installed and working
split(' ',$config{'clamscan_cmd'},2)
&amp;has_command($_[[0]]) || [etc..etc..&gt;

----

sample test of code:

#!/usr/bin/perl
use strict;
use warnings;

my $string = 'clamdscan --config-file /etc/clamd.d/CLAMSPAM.conf';
@_ = split(' ',$string,2);
print $_[[0]];
print &quot;n&quot;;
exit 0;

Joe,

In /usr/libexec/webmin/virtual-server/virtual-server-lib.pl,
there is a test for the &quot;clamscan_cmd&quot; variable:

if ($config{'virus'}) {
# Make sure ClamAV is installed and working
&amp;has_command($config{'clamscan_cmd'}) ||
return &amp;text('index_evirus', &quot;<i>$config{'clamscan_cmd'}</i>&quot;, $clink);
local $out = `$config{'clamscan_cmd'} -[/dev/null 2&gt;&amp;1`;
$? &amp;&amp; return &amp;text('index_evirusrun', &quot;<i>$config{'clamscan_cmd'}</i>&quot;, &quot;[pre]$out[/pre]&quot;, $clink);
&amp;$second_print($text{'check_virusok'});
}

Instead of sending 'clamscan_cmd' to &amp;has_command, couldn't it be split and only the first component of clamscan_cmd be sent to &amp;has_command?

like...

split(' ',$config{'clamscan_cmd'},2)
&amp;has_command($_[[0]]);

if ($config{'virus'}) {
# Make sure ClamAV is installed and working
split(' ',$config{'clamscan_cmd'},2)
&amp;has_command($_[[0]]) || [etc..etc..&gt;

----

sample test of code:

#!/usr/bin/perl
use strict;
use warnings;

my $string = 'clamdscan --config-file /etc/clamd.d/CLAMSPAM.conf';
@_ = split(' ',$string,2);
print $_[[0]];
print &quot;n&quot;;
exit 0;