[PATCH v2 0/4] fix ifunc with static pie [BZ #27072]
Szabolcs Nagy
szabolcs.nagy@arm.com
Mon Jan 11 10:50:57 GMT 2021
The 01/08/2021 09:04, H.J. Lu wrote:
> On Fri, Jan 8, 2021 at 8:22 AM Szabolcs Nagy via Libc-alpha
> <libc-alpha@sourceware.org> wrote:
> >
> > v2:
> > - check PI_STATIC_AND_HIDDEN for --enable-static-pie
> > - change string buffer sizes in the tunables
> > - fix env_alias == NULL logic in __tunables_init
> > - move __ehdr_start processing after self relocation
> >
> > I think this is in a reasonable shape now, but there are still
> > some issues:
> > - tunables try to allocate memory (tunable_strdup) even if
> > that's not necessary: only setuid binaries need this (in
> > case there is a TUNABLE_SECLEVEL_SXID_ERASE tunable).
> > this adds a lot of complexity and a failure path to the early
> > init code. i think that if there is any such tunable then the
> > entire GLIBC_TUNABLE= should just be dropped.
> > - tunable strings could be stored more compactly (and without
> > arbitrary size limits) as an optimization. (can be done once
> > there are too many tunables.)
> > - all symbols are forced hidden in libc.a, but i think lib*.a
> > should do the same. (other than lib*_nonshared.a)
> >
> > Szabolcs Nagy (4):
> > configure: Require PI_STATIC_AND_HIDDEN for static pie
> > Make libc symbols hidden in static PIE
> > elf: Avoid RELATIVE relocs in __tunables_init
> > csu: Move static pie self relocation later [BZ #27072]
> >
>
> Can you push your patches into a branch? I'd like to add an x86 test
> on top of your patches.
i rebased and pushed them into nsz/bug27072
More information about the Libc-alpha
mailing list