jyapayne/musl
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

fix potential race condition in detached threads

Browse source 
after the thread unmaps its own stack/thread structure, the kernel,
performing child tid clear and futex wake, could clobber a new mapping
made at the same location as the just-removed thread's tid field.
disable kernel clearing of child tid to prevent this.
This commit is contained in:
Rich Felker 2012-07-11 23:36:46 -04:00
commit 92f8396b0c
2 changed files with 10 additions and 4 deletions
Download patch file Download diff file Expand all files Collapse all files

10
src/thread/pthread_create.c
View file

@ -36,6 +36,8 @@ void pthread_exit(void *result)
if (!n) exit(0);
if (self->detached && self->map_base) {
if (self->detached == 2)
__syscall(SYS_set_tid_address, 0);
__syscall(SYS_rt_sigprocmask, SIG_BLOCK, (uint64_t[]){-1},0,8);
__unmapself(self->map_base, self->map_size);
}
@ -87,6 +89,7 @@ int pthread_create(pthread_t *res, const pthread_attr_t *attr, void *(*entry)(vo
size_t guard = DEFAULT_GUARD_SIZE;
struct pthread *self = pthread_self(), *new;
unsigned char *map, *stack, *tsd;
unsigned flags = 0x7d8f00;
if (!self) return ENOSYS;
if (!libc.threaded) {
@ -121,7 +124,10 @@ int pthread_create(pthread_t *res, const pthread_attr_t *attr, void *(*entry)(vo
new->start_arg = arg;
new->self = new;
new->tsd = (void *)tsd;
if (attr) new->detached = attr->_a_detach;
if (attr && attr->_a_detach) {
new->detached = 1;
flags -= 0x200000;
}
new->unblock_cancel = self->cancel;
new->canary = self->canary ^ (uintptr_t)&new;
stack = (void *)new;
@ -129,7 +135,7 @@ int pthread_create(pthread_t *res, const pthread_attr_t *attr, void *(*entry)(vo
__synccall_lock();
a_inc(&libc.threads_minus_1);
ret = __clone(start, stack, 0x7d8f00, new, &new->tid, new, &new->tid);
ret = __clone(start, stack, flags, new, &new->tid, new, &new->tid);
__synccall_unlock();

4
src/thread/pthread_detach.c
View file

@ -5,7 +5,7 @@ int pthread_detach(pthread_t t)
/* Cannot detach a thread that's already exiting */
if (a_swap(&t->exitlock, 1))
return pthread_join(t, 0);
t->detached = 1;
t->exitlock = 0;
t->detached = 2;
a_store(&t->exitlock, 0);
return 0;
}