* Re: [PATCH v4 4/6] binfmt_elf: Use elf_load() for library
[not found] ` <CAKbZUD1ojuNN_+x6gkxEMsmLOd5KbCs-wfJcMM==b8+k8_uD_w@mail.gmail.com>
@ 2023-09-29 17:06 ` Kees Cook
0 siblings, 0 replies; only message in thread
From: Kees Cook @ 2023-09-29 17:06 UTC (permalink / raw)
To: Pedro Falcato
Cc: Eric Biederman, Alexander Viro, Christian Brauner, linux-fsdevel,
linux-mm, Sebastian Ott, Thomas Weißschuh, Andrew Morton,
linux-kernel, linux-hardening, linux-arch
On Fri, Sep 29, 2023 at 01:12:13PM +0100, Pedro Falcato wrote:
> On Fri, Sep 29, 2023 at 4:24 AM Kees Cook <keescook@chromium.org> wrote:
> >
> > While load_elf_library() is a libc5-ism, we can still replace most of
> > its contents with elf_load() as well, further simplifying the code.
>
> While I understand you want to break as little as possible (as the ELF
> loader maintainer), I'm wondering if we could axe CONFIG_USELIB
> altogether? Since CONFIG_BINFMT_AOUT also got axed. Does this have
> users anywhere?
I can't even find a libc5 image I can test. :P
I made it non-default in '22:
7374fa33dc2d ("init/Kconfig: remove USELIB syscall by default")
I'm not sure we can drop it entirely, though.
--
Kees Cook
^ permalink raw reply [flat|nested] only message in thread