* Lock remaining m_rpcHandlers use since these accesses are not guaranteed to be thread safe

2009-02-12 18:54:48 +00:00 · 2009-02-12 18:54:48 +00:00 · 6db1f3c6a6
parent 8ed9c3b8cd
commit 6db1f3c6a6
1 changed files with 8 additions and 2 deletions
--- a/OpenSim/Framework/Servers/BaseHttpServer.cs
+++ b/OpenSim/Framework/Servers/BaseHttpServer.cs
@ -605,9 +605,15 @@ namespace OpenSim.Framework.Servers
                {
                    xmlRprcRequest.Params.Add(request.RemoteIPEndPoint); // Param[1]
                    XmlRpcResponse xmlRpcResponse;
-
+                    
                    XmlRpcMethod method;
-                    if (m_rpcHandlers.TryGetValue(methodName, out method))
+                    bool methodWasFound;
+                    lock (m_rpcHandlers)
+                    {
+                        methodWasFound = m_rpcHandlers.TryGetValue(methodName, out method);
+                    }                            
+                    
+                    if (methodWasFound)
                    {
                        xmlRprcRequest.Params.Add(request.Url); // Param[2]