Work around for ClamAV out of date issue?

I know the clamav out of date thing has been covered many times before but I can't find any suggested work around.

Could there be a work around for the out of date warning? Maybe something in ISPConfig that will parse the warning and ignore it if the "WARNING: Local version: x.xx.x Recommended version: 0.xx.x" line is the only error found?

It's a real pain to keep checking what the warning is just to make sure there are no others.

Otherwise, is there a way to install clamav from source without upsetting ISPConfig?

 

Great.

It would be nice and tidy if the message was just ignored as it is a common issue

Great work Till.

 

It should help, if you change line 1616 in server/lib/classes/monitor_tools.inc.php from

Code:

$state = $this->_setState($state, 'info');

to

Code:

$state = $this->_setState($state, 'ok');

But keep in mind, that you will get no warnings in ispconfig if your clamd is outdated.

Since the most distros need a long time to update clamav i didn´t use apt or zypper to install clamav. i build it from source.

 

I don't have that line in my file.

Using ISPConfig V3.0.5.3

 

Sorry, I used my patched file. Look at line 1579.

 

Yup, that's done the job

Much better.

 

Thanks Till, great work.

Is there an idiots guide on how to update to the git version?

 

Hi Till,

I think checking daily.cvd is a better solution. The lastest main.cvd was released in 2013. Only the daily.cvd is updated and main.cvd stays at it is. This file changes for major updates only.

 

I'm now running 3.0.5.4p1 and still have the warning showing. Was the change dropped or do I have something else wrong?

 

Much appreciated.

I know it's just a notice but it is annoying!

 

Looks like this change has also been lost in 3.0.5.4p5.

Any chance of getting it put back?

 

Went to edit the monitor_tools.inc.php file and found this...

Code:

//* Warn when clamav is outdated and main.cld update failed.
                if($clamav_outdated_warning == true && $clamav_bytecode_updated == false) {
                        $state = $this->_setState($state, 'info');
                }

                /*
                 * Return the Result
                 */

I assume that it's got an eronious '*' on the first line and that is should be this...

Code:

// Warn when clamav is outdated and main.cld update failed.
                if($clamav_outdated_warning == true && $clamav_bytecode_updated == false) {
                        $state = $this->_setState($state, 'info');
                }

                /*
                 * Return the Result
                 */

Is that right?

 

I've got it working by doing this in monitor_tools.inc.php

Code:

                 /*
                 * Now we have the last log in the array.
                 * Check if the outdated-string is found...
                 */
                $clamav_outdated_warning = false;
                $clamav_bytecode_updated = false;
                foreach ($lastLog as $line) {
                        if (stristr($line,'outdated')) {
                                $clamav_outdated_warning = true;
                        }
                        // Changed to look at daily.cld insted of main.cld
                        if(stristr($line,'daily.cld is up to date')) {
                                $clamav_bytecode_updated = true;
                        }

                }

                // Warn when clamav is outdated and daily.cld update failed.
                if($clamav_outdated_warning == true && $clamav_bytecode_updated == false) {
                        $state = $this->_setState($state, 'info');
                } else {
                        $state = $this->_setState($state, 'ok');
                }

                /*
                 * Return the Result
                 */