commit c68c4b15f0bed4ff6c590b1cd477deb26be610c1
author Ulrich Weigand <weigand@informatik.uni-erlangen.de> Wed Mar 17 15:14:31 1999 +0000
committer Alexandre Julliard <julliard@winehq.org> Wed Mar 17 15:14:31 1999 +0000
tree f1b7b180c977e3c7806febaef29802098cd0a4e5
parent a841fec257a2730231af68506783c28194d2bf9e

Inter-thread SendMessage() bugfixes:
- Insert new message to be received at the *end* of the SM_PENDING_LIST.
- Do *not* process received messages in ReplyMessage().
- Clear the QS_SMRESULT flag only immediatedly before waiting.

windows/queue.c

diff --git a/windows/queue.c b/windows/queue.c
index 5ed0024..adf918b 100644
--- a/windows/queue.c
+++ b/windows/queue.c
@@ -750,15 +750,22 @@
             break;
             
         case SM_PENDING_LIST:
+        {
             /* make it thread safe, could be accessed by the sender and
              receiver thread */
+            SMSG **prev;
 
             EnterCriticalSection( &queue->cSection );
-            smsg->nextPending = queue->smPending;
-            queue->smPending = smsg;
-            QUEUE_SetWakeBit( queue, QS_SENDMESSAGE );
+            smsg->nextPending = NULL;
+            prev = &queue->smPending;
+            while ( *prev )
+                prev = &(*prev)->nextPending;
+            *prev = smsg;
             LeaveCriticalSection( &queue->cSection );
+
+            QUEUE_SetWakeBit( queue, QS_SENDMESSAGE );
             break;
+        }
 
         default:
             WARN(sendmsg, "Invalid list: %d", list);