fix visibility mismatch in dynamic linker stage 2 function definition
authorRich Felker <dalias@aerifal.cx>
Thu, 15 Oct 2015 21:38:54 +0000 (17:38 -0400)
committerRich Felker <dalias@aerifal.cx>
Thu, 15 Oct 2015 21:38:54 +0000 (17:38 -0400)
commitbc9b6ea0df6a820878d9ac538b2b852fec367d41
tree6f966b7886387368e9c8b72eb8dc074866bd30a0
parent797899802d5623a6ec05663c997a30e03dc27421
fix visibility mismatch in dynamic linker stage 2 function definition

since commits 2907afb8dbd4c1d34825c3c9bd2b41564baca210 and
6fc30c2493fcfedec89e45088bea87766a1e3286, __dls2 is no longer called
via symbol lookup, but instead uses relative addressing that needs to
be resolved at link time. on some linker versions, and/or if
-Bsymbolic-functions is not used, the linker may leave behind a
dynamic relocation, which is not suitable for bootstrapping the
dynamic linker, if the reference to __dls2 is marked hidden but the
definition is not actually hidden. correcting the definition to use
hidden visibility fixes the problem.

the static-PIE entry point rcrt1 was likewise affected and is also
fixed by this patch.
crt/rcrt1.c
src/ldso/dynlink.c