generic-library/bionic
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

am eab2889e: am fd955033: Merge "pthread: Invalidate stale stack pointers on pthread_exit()"

Browse Source 
* commit 'eab2889e66d4fe03f3c5590d7e8b14e3777179ec':
  pthread: Invalidate stale stack pointers on pthread_exit()
This commit is contained in:
Elliott Hughes 2012-05-09 12:56:15 -07:00 committed by Android Git Automerger
commit 22167aafba
1 changed files with 14 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

16
libc/bionic/pthread.c
View File

@ -577,6 +577,17 @@ void pthread_exit(void * retval)
_pthread_internal_remove(thread); _pthread_internal_remove(thread);
_pthread_internal_free(thread); _pthread_internal_free(thread);
} else { } else {
pthread_mutex_lock(&gThreadListLock);
/* make sure that the thread struct doesn't have stale pointers to a stack that
* will be unmapped after the exit call below.
*/
if (!user_stack) {
thread->attr.stack_base = NULL;
thread->attr.stack_size = 0;
thread->tls = NULL;
}
/* the join_count field is used to store the number of threads waiting for /* the join_count field is used to store the number of threads waiting for
* the termination of this thread with pthread_join(), * the termination of this thread with pthread_join(),
* *
@ -589,7 +600,6 @@ void pthread_exit(void * retval)
* is gone (as well as its TLS area). when another thread calls pthread_join() * is gone (as well as its TLS area). when another thread calls pthread_join()
* on it, it will immediately free the thread and return. * on it, it will immediately free the thread and return.
*/ */
pthread_mutex_lock(&gThreadListLock);
thread->return_value = retval; thread->return_value = retval;
if (thread->join_count > 0) { if (thread->join_count > 0) {
pthread_cond_broadcast(&thread->join_cond); pthread_cond_broadcast(&thread->join_cond);
@ -1942,7 +1952,9 @@ int pthread_key_delete(pthread_key_t key)
* similarly, it is possible to have thr->tls == NULL for threads that * similarly, it is possible to have thr->tls == NULL for threads that
* were just recently created through pthread_create() but whose * were just recently created through pthread_create() but whose
* startup trampoline (__thread_entry) hasn't been run yet by the * startup trampoline (__thread_entry) hasn't been run yet by the
* scheduler. so check for this too. * scheduler. thr->tls will also be NULL after it's stack has been
* unmapped but before the ongoing pthread_join() is finished.
* so check for this too.
*/ */
if (thr->join_count < 0 || !thr->tls) if (thr->join_count < 0 || !thr->tls)
continue; continue;