From 1e6f9f0dd924ff977972a42a25711cef44a685fd Mon Sep 17 00:00:00 2001
From: Volker Fischer <corrados@users.noreply.github.com>
Date: Sun, 17 Aug 2008 21:06:25 +0000
Subject: [PATCH] bug fix -> deadlock

---
 src/protocol.cpp | 18 ++++++++++--------
 1 file changed, 10 insertions(+), 8 deletions(-)

diff --git a/src/protocol.cpp b/src/protocol.cpp
index fa0c83cd..ab9c7c8b 100755
--- a/src/protocol.cpp
+++ b/src/protocol.cpp
@@ -164,20 +164,22 @@ void CProtocol::EnqueueMessage ( CVector<uint8_t>& vecMessage,
 
 void CProtocol::SendMessage()
 {
-    QMutexLocker locker ( &Mutex );
-
     CVector<uint8_t> vecMessage;
     bool             bSendMess = false;
 
-    // we have to check that list is not empty, since in another thread the
-    // last element of the list might have been erased
-    if ( !SendMessQueue.empty() )
+    Mutex.lock();
     {
-        vecMessage.Init ( SendMessQueue.front().vecMessage.Size() );
-        vecMessage = SendMessQueue.front().vecMessage;
+        // we have to check that list is not empty, since in another thread the
+        // last element of the list might have been erased
+        if ( !SendMessQueue.empty() )
+        {
+            vecMessage.Init ( SendMessQueue.front().vecMessage.Size() );
+            vecMessage = SendMessQueue.front().vecMessage;
 
-        bSendMess = true;
+            bSendMess = true;
+        }
     }
+    Mutex.unlock();
 
     if ( bSendMess )
     {