From f301ec162bf979f35c108343bd14161540494eb2 Mon Sep 17 00:00:00 2001 From: Ulrich Drepper Date: Thu, 16 Mar 2000 21:29:42 +0000 Subject: [PATCH] (__pthread_mutex_lock): Always initialize __m_owner. (__pthread_mutex_trylock): Likewise. (__pthread_mutex_unlock): Always clear __m_owner. --- linuxthreads/mutex.c | 16 ++++++++-------- 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/linuxthreads/mutex.c b/linuxthreads/mutex.c index 06d97df03ff..f35cab8ae67 100644 --- a/linuxthreads/mutex.c +++ b/linuxthreads/mutex.c @@ -50,6 +50,7 @@ int __pthread_mutex_trylock(pthread_mutex_t * mutex) switch(mutex->__m_kind) { case PTHREAD_MUTEX_FAST_NP: retcode = __pthread_trylock(&mutex->__m_lock); + mutex->__m_owner = thread_self(); return retcode; case PTHREAD_MUTEX_RECURSIVE_NP: self = thread_self(); @@ -77,31 +78,29 @@ strong_alias (__pthread_mutex_trylock, pthread_mutex_trylock) int __pthread_mutex_lock(pthread_mutex_t * mutex) { - pthread_descr self; + pthread_descr self = thread_self(); switch(mutex->__m_kind) { case PTHREAD_MUTEX_FAST_NP: __pthread_lock(&mutex->__m_lock, NULL); - return 0; + break; case PTHREAD_MUTEX_RECURSIVE_NP: - self = thread_self(); if (mutex->__m_owner == self) { mutex->__m_count++; return 0; } __pthread_lock(&mutex->__m_lock, self); - mutex->__m_owner = self; mutex->__m_count = 0; - return 0; + break; case PTHREAD_MUTEX_ERRORCHECK_NP: - self = thread_self(); if (mutex->__m_owner == self) return EDEADLK; __pthread_lock(&mutex->__m_lock, self); - mutex->__m_owner = self; - return 0; + break; default: return EINVAL; } + mutex->__m_owner = self; + return 0; } strong_alias (__pthread_mutex_lock, pthread_mutex_lock) @@ -110,6 +109,7 @@ int __pthread_mutex_unlock(pthread_mutex_t * mutex) switch (mutex->__m_kind) { case PTHREAD_MUTEX_FAST_NP: __pthread_unlock(&mutex->__m_lock); + mutex->__m_owner = NULL; return 0; case PTHREAD_MUTEX_RECURSIVE_NP: if (mutex->__m_count > 0) { -- 2.47.2