sherpya at netfarm
Dec 20, 2007, 4:49 AM
Post #2 of 2
(261 views)
Permalink
|
|
Re: Error building klamav WAS: Error building Mail::ClamAV after clamav ABI change
[In reply to]
|
|
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1
Renato Botelho wrote:
> On Dec 18, 2007 4:06 PM, Gianluigi Tiesi <sherpya[at]netfarm.it> wrote:
>> -----BEGIN PGP SIGNED MESSAGE-----
>> Hash: SHA1
>>
>>
>> Renato Botelho wrote:
>>> On Dec 18, 2007 2:36 PM, Gianluigi Tiesi <sherpya[at]netfarm.it> wrote:
>>>> -----BEGIN PGP SIGNED MESSAGE-----
>>>> Hash: SHA1
>>>>
>>>> I'm not expert using perl
>>>> and this module uses a strange way
>>>> to wrap the c api
>>>> As I see the error code handling
>>>> is wrong by design, you may fix it
>>>> by removing the offending defines from ClamAV.pm
>>> The last problem i have now is to build klamav. It stops to build with
>>> an "undefined reference to `cli_gentempdesc'". Was this function
>>> moved, could i change it to another one?
>>>
>>> Thanks
>> at the top of the file, if c:
>>
>> extern int cli_gentempfd(const char *dir, char **name, int *fd);
>>
>> if cpp:
>> extern "C" int cli_gentempfd(const char *dir, char **name, int *fd);
>>
>> where is the reference to cli_gentempdesc
>> char *name;
>> cli_gentempfd(tmpdir, &name, &fd);
>>
>> yes it still use internals stuff but this is a workaround
>> not a solution you should complain to package author :)
>
> Do you believe it's a correct workaround?
>
> --- src/klamd/scanner.c.orig 2007-12-19 08:33:50.000000000 -0200
> +++ src/klamd/scanner.c 2007-12-19 08:36:04.000000000 -0200
> @@ -396,7 +396,7 @@
>
> logg("*Accepted connection on port %d, fd %d\n", port, acceptd);
>
> - if ((tmpname = cli_gentempdesc(NULL, &tmpd)) == NULL) {
> + if (cli_gentempfd(NULL, &tmpname, &tmpd) ) {
> shutdown(sockfd, 2);
> close(sockfd);
> close(acceptd);
> --- src/klamd/scanner.h.orig 2007-12-19 08:34:32.000000000 -0200
> +++ src/klamd/scanner.h 2007-12-19 08:34:52.000000000 -0200
> @@ -33,4 +33,6 @@
>
> int checksymlink(const char *path);
>
> +extern int cli_gentempfd(const char *dir, char **name, int *fd);
> +
> #endif
>
>
forgot to cc the list:
hmmo cli_gentmpfd it's an internal function too
you should write a custom code that opens a temp file
Regards
- --
Gianluigi Tiesi <sherpya[at]netfarm.it>
EDP Project Leader
Netfarm S.r.l. - http://www.netfarm.it/
Free Software: http://oss.netfarm.it/
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.1 (MingW32)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org
iD8DBQFHamT03UE5cRfnO04RAqkRAJ4q1v2H7oJwtFmO4DqYimi+IhvXTwCgl7E+
F8a/0mKlt+vLoU0Cg/kXofk=
=BnJI
-----END PGP SIGNATURE-----
_______________________________________________
http://lurker.clamav.net/list/clamav-devel.html
Please submit your patches to our Bugzilla: http://bugs.clamav.net
|
