[PATCH] D20024: [ELF] - Protect first entries of got.plt with RelRo.

Rafael EspĂ­ndola via llvm-commits llvm-commits at lists.llvm.org
Fri May 6 12:36:10 PDT 2016


On 6 May 2016 at 14:27, Rafael EspĂ­ndola <rafael.espindola at gmail.com> wrote:
> On 6 May 2016 at 14:23, Rui Ueyama <ruiu at google.com> wrote:
>> Can we simply drop it? I'm wondering if other libc's or dynamic linkers use
>> it.
>
> From what is written in the abi it looks like it would only be used by
> self relocating programs (i.e.: ld.so).
>
> But those could do just what musl does.
>
> Let me give it a try.

http://reviews.llvm.org/D20028

As for the rest of this patch: I think I agree with Rui. The added
value is really small. If going for safety you should really disable
lazy loading or even just compile with -fno-plt.

Cheers,
Rafael


More information about the llvm-commits mailing list