[PATCH] Fix ifunc support with DT_TEXTREL segments (BZ#20480)
Florian Weimer
fweimer@redhat.com
Sat Sep 22 12:14:00 GMT 2018
* Adhemerval Zanella:
> Adam J. Richte <adam_richter2004@yahoo.com>
> Adhemerval Zanella <adhemerval.zanella@linaro.org>
> Fangrui Song <maskray@google.com>
>
> [BZ #20480]
> * config.h.in (CAN_TEXTREL_IFUNC): New define.
> * configure.ac: Add check if linker supports textrel relocation with
> ifunc.
> * elf/dl-reloc.c (_dl_relocate_object): Use all required flags on
> DT_TEXTREL segments, not only PROT_READ and PROT_WRITE.
> * elf/Makefile (ifunc-pie-tests): Add tst-ifunc-textrel.
> (CFLAGS-tst-ifunc-textrel.c): New rule.
> * elf/tst-ifunc-textrel.c: New file.
The code change looks okay to me. Not sure about copyright assignment
needs for this one, but it's probably below the threshold because it's
just moving code.
I would prefer if the new test was disabled at the makefile level, not
with a preprocessor conditional. But it's not a strong preference.
Thanks,
Florian
More information about the Libc-alpha
mailing list