add locking to internal queue in WebFetchInvDescModule; lack of which caused a random crash in a load test yesterday

1 files changed, 10 insertions, 5 deletions

diff --git a/OpenSim/Region/ClientStack/Linden/Caps/WebFetchInvDescModule.cs b/OpenSim/Region/ClientStack/Linden/Caps/WebFetchInvDescModule.cs
index 164adeb..4ff617f 100644
--- a/OpenSim/Region/ClientStack/Linden/Caps/WebFetchInvDescModule.cs
+++ b/OpenSim/Region/ClientStack/Linden/Caps/WebFetchInvDescModule.cs
@@ -254,10 +254,13 @@ namespace OpenSim.Region.ClientStack.Linden
                         }
                     }
 
-                    if (highPriority)
-                        m_queue.EnqueueHigh(reqinfo);
-                    else
-                        m_queue.EnqueueLow(reqinfo);
+                    lock (m_queue)
+                    {
+                        if (highPriority)
+                            m_queue.EnqueueHigh(reqinfo);
+                        else
+                            m_queue.EnqueueLow(reqinfo);
+                    }
                 };
 
                 NoEvents = (x, y) =>
@@ -345,7 +348,9 @@ namespace OpenSim.Region.ClientStack.Linden
             {
                 Watchdog.UpdateThread();
 
-                aPollRequest poolreq = m_queue.Dequeue();
+                aPollRequest poolreq = null;
+                lock (m_queue)
+                    poolreq = m_queue.Dequeue();
 
                 if (poolreq != null && poolreq.thepoll != null)
                     poolreq.thepoll.Process(poolreq);