From: Ulrich Drepper <drepper@redhat.com>
Date: Thu, 16 Mar 2000 21:29:42 +0000 (+0000)
Subject: (__pthread_mutex_lock): Always initialize __m_owner.
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=f301ec162bf979f35c108343bd14161540494eb2;p=thirdparty%2Fglibc.git

(__pthread_mutex_lock): Always initialize __m_owner.
(__pthread_mutex_trylock): Likewise.
(__pthread_mutex_unlock): Always clear __m_owner.
---

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) {