[Pkg-clamav-devel] new clamav 0.98.5 beta1

Andreas Cadhalpun andreas.cadhalpun at googlemail.com
Tue Aug 12 16:17:40 UTC 2014


Hi,

On 12.08.2014 02:17, Scott Kitterman wrote:
> On Monday, August 11, 2014 23:48:24 Andreas Cadhalpun wrote:
>> 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);
>> ^
>
> Did you report this upstream?

No, because I wanted to investigate them first. ;)
It turns out, that two of them are due to the libmspack patch 
(libmspack.h declared the functions with a _lib suffix, but the 
functions in libmspack.c don't have that suffix) and the other two are 
already fixed upstream in the master branch. I added a patch for them.

> I tried building the current package and got test errors (here's an excerpt):

On amd64 there is no problem, but I can reproduce this on i386.
I assumed the libmspack usage wasn't architecture dependent, but 
apparently it is.

Best regards,
Andreas




More information about the Pkg-clamav-devel mailing list