summaryrefslogtreecommitdiff
path: root/src/thread
diff options
context:
space:
mode:
authorRich Felker <dalias@aerifal.cx>2011-04-20 21:41:45 -0400
committerRich Felker <dalias@aerifal.cx>2011-04-20 21:41:45 -0400
commit870cc679771f776333953b2a990a107393d9d0fd (patch)
tree45b6de4ba1170de4654b23d62bd1c6a38220e112 /src/thread
parentf9a6372a98cc4d1b70400b2e7238e1f9eae50558 (diff)
downloadmusl-870cc679771f776333953b2a990a107393d9d0fd.tar.gz
musl-870cc679771f776333953b2a990a107393d9d0fd.tar.bz2
musl-870cc679771f776333953b2a990a107393d9d0fd.tar.xz
musl-870cc679771f776333953b2a990a107393d9d0fd.zip
fix minor bugs due to incorrect threaded-predicate semantics
some functions that should have been testing whether pthread_self() had been called and initialized the thread pointer were instead testing whether pthread_create() had been called and actually made the program "threaded". while it's unlikely any mismatch would occur in real-world problems, this could have introduced subtle bugs. now, we store the address of the main thread's thread descriptor in the libc structure and use its presence as a flag that the thread register is initialized. note that after fork, the calling thread (not necessarily the original main thread) is the new main thread.
Diffstat (limited to 'src/thread')
-rw-r--r--src/thread/pthread_self.c6
-rw-r--r--src/thread/pthread_setcancelstate.c2
2 files changed, 3 insertions, 5 deletions
diff --git a/src/thread/pthread_self.c b/src/thread/pthread_self.c
index e8bb9181..55d20c9f 100644
--- a/src/thread/pthread_self.c
+++ b/src/thread/pthread_self.c
@@ -6,7 +6,6 @@ static struct pthread main_thread;
static const void *dummy[1] = { 0 };
weak_alias(dummy, __pthread_tsd_main);
-#undef errno
static int *errno_location()
{
return __pthread_self()->errno_ptr;
@@ -14,11 +13,10 @@ static int *errno_location()
static int init_main_thread()
{
+ if (__set_thread_area(&main_thread) < 0) return -1;
main_thread.canceldisable = libc.canceldisable;
main_thread.tsd = (void **)__pthread_tsd_main;
- main_thread.self = &main_thread;
- if (__set_thread_area(&main_thread) < 0)
- return -1;
+ main_thread.self = libc.main_thread = &main_thread;
main_thread.errno_ptr = __errno_location();
libc.errno_location = errno_location;
main_thread.tid = main_thread.pid =
diff --git a/src/thread/pthread_setcancelstate.c b/src/thread/pthread_setcancelstate.c
index ebb6eba4..ba2b2311 100644
--- a/src/thread/pthread_setcancelstate.c
+++ b/src/thread/pthread_setcancelstate.c
@@ -3,7 +3,7 @@
int pthread_setcancelstate(int new, int *old)
{
if (new > 1U) return EINVAL;
- if (libc.threaded) {
+ if (libc.main_thread) {
struct pthread *self = __pthread_self();
if (old) *old = self->canceldisable;
self->canceldisable = new;