On Jul 30, 2005, at 03:25 PM, Dale Walsh wrote:


On Jul 30, 2005, at 07:23 AM, Tomasz Kojm wrote:


On Fri, 29 Jul 2005 15:31:52 -0700
James Paige <[EMAIL PROTECTED]> wrote:



Dale Walsh wrote:



On Jul 29, 2005, at 04:45 PM, Tomasz Kojm wrote:



On Fri, 29 Jul 2005 16:23:57 -0400
Dale Walsh <[EMAIL PROTECTED]> wrote:




I've already linked against 0.13.38 and it doesn't appear to have any ill effects on libclamav so maybe what is in 0.13.38 is compatible
with the fixes you implemented in 0.10.27.




No, it isn't. Please read the sources and hopefully you will understand
what I was talking about.




I's catching viruses and doesn't look like there are any problems..




I have no time to explain in which cases your compilation will fail.

--
   oo    .....         Tomasz Kojm <[EMAIL PROTECTED]>





Tomasz, can I get a diff on the changes you implemented in zziplib please?

-- Dale



I know Tomasz is a busy man ;) but if he remembers which version of
zziplib was originally forked for inclusion in clamav, it would not be
difficult for one of us to create the diff.



The current zziplib code in libclamav is based on 0.12.83

--
   oo    .....         Tomasz Kojm <[EMAIL PROTECTED]>


I've generated a diff against the original zziplib you stated.

I'm curious why you commented out the realdir stuff??

Is there a problem having this function that CLamAV doesn't like or was it just to reduce it to used functions??

In it's entirety, zzip uses libzip but you don't seem to use this library or the libzzip-seek functions.

0.13.38 is a little more complex in nature, I'll look into it further but so far I haven't experienced any issues with using an external zziplib and the only other solution I can see to avoid the zziplib conflict would be to build the intenal zziplib as an external libclamav-zzip and have libclamav link against it solving library conflicts.

I'll look further into it.

-- Dale

I've broken zziplib into a seperate library and this seems to help the libzzip conflict but I'm not sure if it will affect libclamav negatively, the change was rather simple to achieve.

Here's what I have so far.

mustangrestomods:~/src/clamav-0.86.2/libclamav/.libs root# otool -L llibclamav.1.0.16.dylib libclamavzzip.12.0.83.dylib
libclamav/.libs/libclamav.1.0.16.dylib:
/usr/local/lib/libclamav.1.dylib (compatibility version 2.0.0, current version 2.16.0) /usr/lib/libbz2.1.0.dylib (compatibility version 1.0.0, current version 1.0.2) /usr/local/lib/libz.1.dylib (compatibility version 1.0.0, current version 1.2.3) /usr/lib/libcrypto.0.9.7.dylib (compatibility version 0.9.7, current version 0.9.7) /usr/lib/libssl.0.9.7.dylib (compatibility version 0.9.7, current version 0.9.7) /usr/local/lib/libmpfr.1.dylib (compatibility version 2.0.0, current version 2.0.0) /usr/local/lib/libgmp.3.dylib (compatibility version 7.0.0, current version 7.3.0) /usr/local/lib/libcurl.3.dylib (compatibility version 4.0.0, current version 4.0.0) /usr/local/lib/libclamavzzip.12.dylib (compatibility version 13.0.0, current version 13.83.0) /usr/lib/libgcc_s.1.dylib (compatibility version 1.0.0, current version 1.0.0) /usr/lib/libmx.A.dylib (compatibility version 1.0.0, current version 92.0.0) /usr/lib/libSystem.B.dylib (compatibility version 1.0.0, current version 88.0.0)
libclamav/.libs/libclamavzzip.12.0.83.dylib:
/usr/local/lib/libclamavzzip.12.dylib (compatibility version 13.0.0, current version 13.83.0) /usr/lib/libbz2.1.0.dylib (compatibility version 1.0.0, current version 1.0.2) /usr/local/lib/libz.1.dylib (compatibility version 1.0.0, current version 1.2.3) /usr/lib/libgcc_s.1.dylib (compatibility version 1.0.0, current version 1.0.0) /usr/lib/libmx.A.dylib (compatibility version 1.0.0, current version 92.0.0) /usr/lib/libSystem.B.dylib (compatibility version 1.0.0, current version 88.0.0)
mustangrestomods:~/src/clamav-0.86.2/libclamav/.libs root#


Now I guess all I need to do is examine the newer zziplib and see what it would take to make it work however, I still don't know what to do about realdir so for now I'll look at what it would take to bypass this function which seems to be the only major change to the included zziplib and there's probably a good reason for this, I just haven't been told what it is yet.


-- Dale

_______________________________________________
http://lurker.clamav.net/list/clamav-devel.html

Reply via email to