remove undef weak refs to init/fini array symbols in libc.so
authorRich Felker <dalias@aerifal.cx>
Fri, 20 Nov 2015 01:28:08 +0000 (20:28 -0500)
committerRich Felker <dalias@aerifal.cx>
Fri, 20 Nov 2015 03:10:55 +0000 (22:10 -0500)
commit19caa25d0a8e587bb89b79c3f629085548709dd4
tree479d4a303ea7a1a0d0efb511fa09d80730a7e513
parent4f3a92881aa1fd9a3ebebe2c9940d70c95a7b0e6
remove undef weak refs to init/fini array symbols in libc.so

commit ad1cd43a86645ba2d4f7c8747240452a349d6bc1 eliminated
preprocessor-level omission of references to the init/fini array
symbols from object files going into libc.so. the references are weak,
and the intent was that the linker would resolve them to zero in
libc.so, but instead it leaves undefined references that could be
satisfied at runtime. normally these references would be harmless,
since the code using them does not even get executed, but some older
binutils versions produce a linking error: when linking a program
against libc.so, ld first tries to use the hidden init/fini array
symbols produced by the linker script to satisfy the references in
libc.so, then produces an error because the definitions are hidden.

ideally ld would have already provided definitions of these symbols
when linking libc.so, but the linker script for -shared omits them.

to avoid this situation, the dynamic linker now provides its own dummy
definitions of the init/fini array symbols for libc.so. since they are
hidden, everything binds at ld time and no references remain in the
dynamic symbol table. with modern binutils and --gc-sections, both
the dummy empty array objects and the code referencing them get
dropped at link time, anyway.

the _init and _fini symbols are also switched back to using weak
definitions rather than weak references since the latter behave
somewhat problematically in general, and the weak definition approach
was known to work well.
src/env/__libc_start_main.c
src/exit/exit.c
src/internal/vis.h
src/ldso/dynlink.c