about summary refs log tree commit diff
path: root/src/thread/pthread_create.c
diff options
context:
space:
mode:
authorRich Felker <dalias@aerifal.cx>2011-02-19 11:04:36 -0500
committerRich Felker <dalias@aerifal.cx>2011-02-19 11:04:36 -0500
commit19eb13b9a4cf2f787f60b6e2a6d26a6cd7d3ffd2 (patch)
treec02fa747b1ad7370b689af34f15139cb62fa9863 /src/thread/pthread_create.c
parenta49c119276742d7d212fb88f83a8f559ca549e72 (diff)
downloadmusl-19eb13b9a4cf2f787f60b6e2a6d26a6cd7d3ffd2.tar.gz
musl-19eb13b9a4cf2f787f60b6e2a6d26a6cd7d3ffd2.tar.xz
musl-19eb13b9a4cf2f787f60b6e2a6d26a6cd7d3ffd2.zip
race condition fix: block all signals before decrementing thread count
the existence of a (kernelspace) thread must never have observable
effects after the thread count is decremented. if signals are not
blocked, it could end up handling the signal for rsyscall and
contributing towards the count of threads which have changed ids,
causing a thread to be missed. this could lead to one thread retaining
unwanted privilege level.

this change may also address other subtle race conditions in
application code that uses signals.
Diffstat (limited to 'src/thread/pthread_create.c')
-rw-r--r--src/thread/pthread_create.c2
1 files changed, 2 insertions, 0 deletions
diff --git a/src/thread/pthread_create.c b/src/thread/pthread_create.c
index 7c436957..c73c5211 100644
--- a/src/thread/pthread_create.c
+++ b/src/thread/pthread_create.c
@@ -25,6 +25,8 @@ void __pthread_unwind_next(struct __ptcb *cb)
 		}
 	}
 
+	syscall4(__NR_sigprocmask, SIG_BLOCK, (long)(uint64_t[1]){-1}, 0, 8);
+
 	if (!a_fetch_add(&libc.threads_minus_1, -1))
 		exit(0);