[Pkg-clamav-devel] new clamav 0.98.5 beta1

[Andreas Cadhalpun]

Hi,

On 10.08.2014 15:38, Sebastian Andrzej Siewior wrote:
> I just pushed the beta1 into the unstable branch. I have to run now and
> I try to add the libmspack later…

The libmspack branch is already merged and I now added the systemd 
patches. Do you intend to also merge the bigeasy branch for this upload?

> Side note: I added
>
> |+++ b/debian/libclamav6.symbols
> |@@ -22,6 +22,7 @@ libclamav.so.6 libclamav6 #MINVER#
> |+ cl_engine_set_clcb_file_props at CLAMAV_PUBLIC 0.98.5~beta1+dfsg-1
> |  cl_engine_set_clcb_hash at CLAMAV_PUBLIC 0.98.1
>
> I remember that dfsg-1 was wrong and the debian version should go away.

Yes, the debian revision should never appear in the symbols file. I 
fixed that.

> However I am uncomfortable adding 0.98.5 since it is not out (yet).
> Should we keep it as-is and string everything past 0.98.5 once it is
> out?

Adding 0.98.5 wouldn't work, because it is larger than 0.98.5~beta1, but 
having only this should be fine. Once the final release is made, we can 
update that to 0.98.5, but we could just as well leave it as is.

Anyway it's good that we test this beta version, because I already found 
a problem that should be fixed before the final release: there are a 
number of implicit function declaration warnings:
scanners.c: In function 'cli_scanraw':
scanners.c:2109:25: warning: implicit declaration of function 
'cli_scanmscab' [-Wimplicit-function-declaration]
                          nret = cli_scanmscab(ctx, fpt->offset);
                          ^
scanners.c:2614:17: warning: implicit declaration of function 
'cli_process_ooxml' [-Wimplicit-function-declaration]
                  ret = cli_process_ooxml(ctx);
                  ^
scanners.c:2727:3: warning: implicit declaration of function 
'cli_scanmschm' [-Wimplicit-function-declaration]
    ret = cli_scanmschm(ctx);
    ^
ooxml.c:464:21: warning: implicit declaration of function 
'unzip_single_internal' [-Wimplicit-function-declaration]
                      ret = unzip_single_internal(ctx, loff, ooxml_core_cb);
                      ^

Best regards,
Andreas