linux-kernel - Re: [PATCH 20/20] tools lib: Fix builds when glibc contains strlcpy()

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:   Tue, 7 Jan 2020 23:48:04 +0300
From:   Vitaly Chikunov <vt@...linux.org>
To:     David Laight <David.Laight@...LAB.COM>
Cc:     'Arnaldo Carvalho de Melo' <acme@...nel.org>,
        Ingo Molnar <mingo@...nel.org>,
        Thomas Gleixner <tglx@...utronix.de>,
        Jiri Olsa <jolsa@...nel.org>,
        Namhyung Kim <namhyung@...nel.org>,
        Clark Williams <williams@...hat.com>,
        "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
        "linux-perf-users@...r.kernel.org" <linux-perf-users@...r.kernel.org>,
        Dmitry Levin <ldv@...linux.org>,
        Josh Poimboeuf <jpoimboe@...hat.com>,
        kbuild test robot <lkp@...el.com>,
        Peter Zijlstra <peterz@...radead.org>,
        "stable@...r.kernel.org" <stable@...r.kernel.org>,
        Vineet Gupta <vineet.gupta1@...opsys.com>,
        Arnaldo Carvalho de Melo <acme@...hat.com>
Subject: Re: [PATCH 20/20] tools lib: Fix builds when glibc contains strlcpy()

David,

On Mon, Jan 06, 2020 at 05:03:26PM +0000, David Laight wrote:
> From: Arnaldo Carvalho de Melo
> > Sent: 06 January 2020 16:07
> > 
> > From: Vitaly Chikunov <vt@...linux.org>
> > 
> > Disable a couple of compilation warnings (which are treated as errors)
> > on strlcpy() definition and declaration, allowing users to compile perf
> > and kernel (objtool) when:
> > 
> > 1. glibc have strlcpy() (such as in ALT Linux since 2004) objtool and
> >    perf build fails with this (in gcc):
> > 
> >   In file included from exec-cmd.c:3:
> >   tools/include/linux/string.h:20:15: error: redundant redeclaration of ‘strlcpy’ [-Werror=redundant-decls]
> >      20 | extern size_t strlcpy(char *dest, const char *src, size_t size);
> > 
> > 2. clang ignores `-Wredundant-decls', but produces another warning when
> >    building perf:
> > 
> >     CC       util/string.o
> >   ../lib/string.c:99:8: error: attribute declaration must precede definition [-Werror,-Wignored-attributes]
> >   size_t __weak strlcpy(char *dest, const char *src, size_t size)
> >   ../../tools/include/linux/compiler.h:66:34: note: expanded from macro '__weak'
> >   # define __weak                 __attribute__((weak))
> >   /usr/include/bits/string_fortified.h:151:8: note: previous definition is here
> >   __NTH (strlcpy (char *__restrict __dest, const char *__restrict __src,
> 
> Why not just always use the local version by renaming it?

I believe that was the initial approach, which is changed some time ago
to using weak linking. Also, Dmitry Levin (as one of glibc maintainers)
claims that glibc implementation of strlcpy() is more correct and thus
better.

Thanks,

> 
> 	David
> 
> -
> Registered Address Lakeside, Bramley Road, Mount Farm, Milton Keynes, MK1 1PT, UK
> Registration No: 1397386 (Wales)