From: Lars Op den Kamp <lars@opdenkamp.eu>
Date: Tue, 21 Feb 2012 12:40:43 +0000 (+0100)
Subject: platform: fixed accidental double deref. the method parameter already does that.... 
X-Git-Tag: upstream/2.2.0~1^2~33^2~38
X-Git-Url: https://git.piment-noir.org/?p=deb_libcec.git;a=commitdiff_plain;h=8f6e48cd4aaefa12c0a5cbd6cc17f8649396bdf6

platform: fixed accidental double deref. the method parameter already does that. closes #24
---

diff --git a/src/lib/platform/threads/mutex.h b/src/lib/platform/threads/mutex.h
index 6bd126e..ed60ba8 100644
--- a/src/lib/platform/threads/mutex.h
+++ b/src/lib/platform/threads/mutex.h
@@ -343,11 +343,11 @@ namespace PLATFORM
       return bReturn;
     }
 
-    volatile bool              m_bSignaled;
-    CCondition<volatile bool&> m_condition;
-    CMutex                     m_mutex;
-    volatile bool              m_bBroadcast;
-    unsigned int               m_iWaitingThreads;
-    bool                       m_bAutoReset;
+    volatile bool             m_bSignaled;
+    CCondition<volatile bool> m_condition;
+    CMutex                    m_mutex;
+    volatile bool             m_bBroadcast;
+    unsigned int              m_iWaitingThreads;
+    bool                      m_bAutoReset;
   };
 }
diff --git a/src/lib/platform/threads/threads.h b/src/lib/platform/threads/threads.h
index f2de310..0efebdc 100644
--- a/src/lib/platform/threads/threads.h
+++ b/src/lib/platform/threads/threads.h
@@ -133,11 +133,11 @@ namespace PLATFORM
     void SetRunning(bool bSetTo);
 
   private:
-    bool               m_bStop;
-    bool               m_bRunning;
-    bool               m_bStopped;
-    CCondition<bool &> m_threadCondition;
-    CMutex             m_threadMutex;
-    thread_t           m_thread;
+    bool             m_bStop;
+    bool             m_bRunning;
+    bool             m_bStopped;
+    CCondition<bool> m_threadCondition;
+    CMutex           m_threadMutex;
+    thread_t         m_thread;
   };
 };