about summary refs log tree commit diff
path: root/src/internal/pthread_impl.h
diff options
context:
space:
mode:
authorRich Felker <dalias@aerifal.cx>2011-06-14 01:35:51 -0400
committerRich Felker <dalias@aerifal.cx>2011-06-14 01:35:51 -0400
commit7779dbd2663269b465951189b4f43e70839bc073 (patch)
treefa131e2ddcf164bf4d58a07f627a4704a0b13f73 /src/internal/pthread_impl.h
parentf58c8a0f391987a65e055ae591ec59b9df7b7f7c (diff)
downloadmusl-7779dbd2663269b465951189b4f43e70839bc073.tar.gz
musl-7779dbd2663269b465951189b4f43e70839bc073.tar.xz
musl-7779dbd2663269b465951189b4f43e70839bc073.zip
fix race condition in pthread_kill
if thread id was reused by the kernel between the time pthread_kill
read it from the userspace pthread_t object and the time of the tgkill
syscall, a signal could be sent to the wrong thread. the tgkill
syscall was supposed to prevent this race (versus the old tkill
syscall) but it can't; it can only help in the case where the tid is
reused in a different process, but not when the tid is reused in the
same process.

the only solution i can see is an extra lock to prevent threads from
exiting while another thread is trying to pthread_kill them. it should
be very very cheap in the non-contended case.
Diffstat (limited to 'src/internal/pthread_impl.h')
-rw-r--r--src/internal/pthread_impl.h1
1 files changed, 1 insertions, 0 deletions
diff --git a/src/internal/pthread_impl.h b/src/internal/pthread_impl.h
index 12f8ccfc..2089c857 100644
--- a/src/internal/pthread_impl.h
+++ b/src/internal/pthread_impl.h
@@ -46,6 +46,7 @@ struct pthread {
 	int unblock_cancel;
 	int delete_timer;
 	locale_t locale;
+	int killlock;
 };
 
 struct __timer {