Hmm, that's a bit odd.
Could you send your configure log so I could take a better look at what's
going on here?
The "_strndup" also strikes me as strange. I would expect to see a symbol
not found failure for "strndup" or "cli_strndup", but "_strndup" is a new
one. Did everything patch okay?
-Mickey
On Fri, Aug 25, 2017 at 10:41 AM, Mark Allan <markjal...@gmail.com> wrote:
> Hi Mickey,
>
> No, it's not working I'm afraid. I get the same error as before:
>
> dyld: lazy symbol binding failed: Symbol not found: _strndup
> Referenced from: /usr/local/clamav/sbin/clamd
> Expected in: /usr/lib/libSystem.B.dylib
>
> Note that this *not* being built on 10.6. It's being built on 10.12 with
> support for running the compiled binaries on 10.6 by way of the
> -mmacosx-version-min=10.6 compiler flag.
>
> Cheers
> Mark
>
> > On 23 Aug 2017, at 7:15 pm, Mickey Sola <ms...@sourcefire.com> wrote:
> >
> > Hi Mark,
> >
> > The strnlen and strndup reworks have made it up to master if you wanted
> to
> > take a look and make sure everything builds OK on 10.6
> >
> > You'll need commits 47a544dc07b75c284e0fc475164bcdc5e9d5b18b thru
> > 8cb271e25cf43bd5d6296827d2c0f25a33420fd9
> > (4 in total)
> >
> > -Mickey
> >
> > On Mon, Aug 14, 2017 at 1:41 PM, Steven Morgan <smor...@sourcefire.com>
> > wrote:
> >
> >> Mark,
> >>
> >> We are in the process of reworking that strndup/strnlen test. The rework
> >> will use feature tests during ./configure to test for the presence of
> the
> >> system implementations of strndup and strnlen. The operating system test
> >> that is currently in place for when to use the local implementations of
> >> strnlen and strndup will be going away. Thanks for writing a patch. It
> >> should suffice during beta.
> >>
> >>
> >> Steve
> >>
> >>
> >> On Mon, Aug 14, 2017 at 9:47 AM, Mark Allan <markjal...@gmail.com>
> wrote:
> >>
> >>> I just had another look at this today with fresh eyes and I see you've
> >>> already got a static replacement of strndup for Solaris, so I've
> >> included a
> >>> patch which uses the same function on macOS 10.6.8 or lower. It relies
> >> on
> >>> the appropriate (-mmacosx-version-min=10.6) setting on the configure
> >>> phase, but the chances are if anyone's compiling with 10.6 support,
> they
> >>> probably ain't compiling on 10.6 so it's likely being supplied already.
> >>>
> >>>
> >>>
> >>>
> >>> diff -Naurw clamav-0.99.3-beta1/clamd/localserver.c
> >>> clamav-0.99.3-beta1_patched/clamd/localserver.c
> >>> --- clamav-0.99.3-beta1/clamd/localserver.c 2017-07-31
> >>> 19:34:32.000000000 +0100
> >>> +++ clamav-0.99.3-beta1_patched/clamd/localserver.c 2017-08-14
> >>> 14:24:08.000000000 +0100
> >>> @@ -25,7 +25,7 @@
> >>>
> >>> #include <stdio.h>
> >>> #include <string.h>
> >>> -#if defined(C_SOLARIS)
> >>> +#if defined(C_SOLARIS) || (defined(__ENVIRONMENT_MAC_OS_
> >> X_VERSION_MIN_REQUIRED__)
> >>> && (__ENVIRONMENT_MAC_OS_X_VERSION_MIN_REQUIRED__ <= 1068))
> >>> size_t strnlen(const char *s, size_t n) __attribute__((weak));
> >>> size_t strnlen(const char *s, size_t n)
> >>> {
> >>>
> >>>
> >>>
> >>> Hope that's useful.
> >>>
> >>> Mark
> >>>
> >>>
> >>>> On 13 Aug 2017, at 10:25 pm, Mark Allan <markjal...@gmail.com> wrote:
> >>>>
> >>>> Hi all,
> >>>>
> >>>> Another issue with 0.99.3 beta 1.
> >>>>
> >>>> The clamd process crashes on macOS 10.6.8 because it can't find the
> >>> strndup symbol. There are a couple of references to strndup in the
> >> source
> >>> for clamd and libclamav - should these be changed to cli_strndup or am
> I
> >>> better to include a static replacement function of strndup in the
> >>> appropriate files that would only be used on 10.6 or earlier?
> >>>>
> >>>> Thanks
> >>>> Mark
>
>
_______________________________________________
clamav-users mailing list
clamav-users@lists.clamav.net
http://lists.clamav.net/cgi-bin/mailman/listinfo/clamav-users
Help us build a comprehensive ClamAV guide:
https://github.com/vrtadmin/clamav-faq
http://www.clamav.net/contact.html#ml
