]> git.ipfire.org Git - thirdparty/glibc.git/commit
elf: Support recursive use of dynamic TLS in interposed malloc
authorFlorian Weimer <fweimer@redhat.com>
Mon, 1 Jul 2024 15:42:04 +0000 (17:42 +0200)
committerSunil K Pandey <skpgkp2@gmail.com>
Sat, 11 Jan 2025 17:30:26 +0000 (09:30 -0800)
commitc3beedeb7010456803e5bc2029761c73b514e97d
tree72a7688d3a44876fd18827620bca843f2cbfd8f4
parentf48d763ab8d1005000b49bcdcd5bb51641b78f85
elf: Support recursive use of dynamic TLS in interposed malloc

It turns out that quite a few applications use bundled mallocs that
have been built to use global-dynamic TLS (instead of the recommended
initial-exec TLS).  The previous workaround from
commit afe42e935b3ee97bac9a7064157587777259c60e ("elf: Avoid some
free (NULL) calls in _dl_update_slotinfo") does not fix all
encountered cases unfortunatelly.

This change avoids the TLS generation update for recursive use
of TLS from a malloc that was called during a TLS update.  This
is possible because an interposed malloc has a fixed module ID and
TLS slot.  (It cannot be unloaded.)  If an initially-loaded module ID
is encountered in __tls_get_addr and the dynamic linker is already
in the middle of a TLS update, use the outdated DTV, thus avoiding
another call into malloc.  It's still necessary to update the
DTV to the most recent generation, to get out of the slow path,
which is why the check for recursion is needed.

The bookkeeping is done using a global counter instead of per-thread
flag because TLS access in the dynamic linker is tricky.

All this will go away once the dynamic linker stops using malloc
for TLS, likely as part of a change that pre-allocates all TLS
during pthread_create/dlopen.

Fixes commit d2123d68275acc0f061e73d5f86ca504e0d5a344 ("elf: Fix slow
tls access after dlopen [BZ #19924]").

Reviewed-by: Szabolcs Nagy <szabolcs.nagy@arm.com>
(cherry picked from commit 018f0fc3b818d4d1460a4e2384c24802504b1d20)
elf/Makefile
elf/dl-tls.c
elf/rtld.c
elf/tst-recursive-tls.c [new file with mode: 0644]
elf/tst-recursive-tlsmallocmod.c [new file with mode: 0644]
elf/tst-recursive-tlsmodN.c [new file with mode: 0644]
sysdeps/generic/ldsodefs.h
sysdeps/x86_64/dl-tls.c