diff options
author | kib <kib@FreeBSD.org> | 2015-01-03 18:38:46 +0000 |
---|---|---|
committer | kib <kib@FreeBSD.org> | 2015-01-03 18:38:46 +0000 |
commit | 8d1dfb4106d37750d526e6b8edca5945f8b0a4d5 (patch) | |
tree | e381919bf3f754392eb9a47b256135a1dbb2b2b1 /lib/libc/sys/__error.c | |
parent | 7ac15f4b4cebad0f4e9d2094b6ea3fefa77ae568 (diff) | |
download | FreeBSD-src-8d1dfb4106d37750d526e6b8edca5945f8b0a4d5.zip FreeBSD-src-8d1dfb4106d37750d526e6b8edca5945f8b0a4d5.tar.gz |
Fix known issues which blow up the process after dlopen("libthr.so")
(or loading a dso linked to libthr.so into process which was not
linked against threading library).
- Remove libthr interposers of the libc functions, including
__error(). Instead, functions calls are indirected through the
interposing table, similar to how pthread stubs in libc are already
done. Libc by default points either to syscall trampolines or to
existing libc implementations. On libthr load, libthr rewrites the
pointers to the cancellable implementations already in libthr. The
interposition table is separate from pthreads stubs indirection
table to not pull pthreads stubs into static binaries.
- Postpone the malloc(3) internal mutexes initialization until libthr
is loaded. This avoids recursion between calloc(3) and static
pthread_mutex_t initialization.
- Reinstall signal handlers with wrapper on libthr load. The
_rtld_is_dlopened(3) is used to avoid useless calls to sigaction(2)
when libthr is statically referenced from the main binary.
In the process, fix openat(2), swapcontext(2) and setcontext(2)
interposing. The libc symbols were exported at different versions
than libthr interposers. Export both libc and libthr versions from
libc now, with default set to the higher version from libthr.
Remove unused and disconnected swapcontext(3) userspace implementation
from libc/gen.
No objections from: deischen
Tested by: pho, antoine (exp-run) (previous versions)
Sponsored by: The FreeBSD Foundation
MFC after: 1 week
Diffstat (limited to 'lib/libc/sys/__error.c')
-rw-r--r-- | lib/libc/sys/__error.c | 15 |
1 files changed, 9 insertions, 6 deletions
diff --git a/lib/libc/sys/__error.c b/lib/libc/sys/__error.c index c3f59f8..329d124 100644 --- a/lib/libc/sys/__error.c +++ b/lib/libc/sys/__error.c @@ -32,14 +32,17 @@ __FBSDID("$FreeBSD$"); extern int errno; -/* - * Declare a weak reference in case the application is not linked - * with libpthread. - */ -__weak_reference(__error_unthreaded, __error); - int * __error_unthreaded(void) { return(&errno); } + +int *(*__error_selector)(void) = __error_unthreaded; + +int * +__error(void) +{ + + return (__error_selector()); +} |