libXfont2 strlcat/strlcpy patch

Roman Gilg subdiff at gmail.com
Fri Dec 6 18:28:07 UTC 2019


On Wed, Nov 20, 2019 at 1:53 AM Guillem Jover <guillem at hadrons.org> wrote:
>
> Hi!
>
> On Sun, 2019-11-03 at 18:08:00 +0100, Roman Gilg wrote:
> > On Wed, Oct 23, 2019 at 5:33 PM Alan Coopersmith wrote:
> > > On 10/23/19 12:48 AM, Michael Joost wrote:
> > > > regarding your recent patch in xorg/libXfont2 (strlcat/strlcpy fallbacks) I ran into a problem on Linux while having libbsd installed. The configure correctly detects libbsd and those functions, however, the macro HAVE_LIBBSD is never defined in this process. The compile then fails with "implicit declaration" error, as the libbsd includes are not loaded from replace.h. When defining this macro manually, everything is fine.
> > > > I think HAVE_LIBBSD should be defined by configure when it has found -lbsd.
> > >
> > > Hmm, I see what you mean, but I've not heard of problems from any other Linux
> > > users/distro builders, so perhaps something else is masking it for them?
> > >
> > > I'm cc'ing the list to see if those who work on Linux see the problem, or
> > > want to provide a patch that they can test more easily than I can.
> >
> > I don't know what needs to be done here, but just wanted to say that I
> > experienced strlcat, strlcpy problems when compiling libXfont master
> > branch on Arch as well.
> >
> > I can not build it from commit 2178c7445a on.
>
> The attached patch should fix this. I'm not sure whether it would be
> better to put the LIBBSD flags into the OS ones, but that might
> require more code shuffling.

Indeed this patch fixes it. Can build with it applied on top of master.

Tested-by: Roman Gilg <subdiff at gmail.com>

> Thanks,
> Guillem


More information about the xorg-devel mailing list