aboutsummaryrefslogtreecommitdiffstatshomepage
path: root/OpenSim/Framework/Servers
diff options
context:
space:
mode:
authorJustin Clark-Casey (justincc)2011-08-22 02:10:45 +0100
committerJustin Clark-Casey (justincc)2011-08-22 02:10:45 +0100
commitc587b0a3a38337fcd23d01e08c9a990abfab0569 (patch)
treeab0902ff313de34be1529cb753385bffc0ce8287 /OpenSim/Framework/Servers
parentimprove locking of m_agentHandlers in BaseHttpServer (diff)
downloadopensim-SC-c587b0a3a38337fcd23d01e08c9a990abfab0569.zip
opensim-SC-c587b0a3a38337fcd23d01e08c9a990abfab0569.tar.gz
opensim-SC-c587b0a3a38337fcd23d01e08c9a990abfab0569.tar.bz2
opensim-SC-c587b0a3a38337fcd23d01e08c9a990abfab0569.tar.xz
oops, fix build break from last commit
Diffstat (limited to 'OpenSim/Framework/Servers')
-rw-r--r--OpenSim/Framework/Servers/HttpServer/BaseHttpServer.cs20
1 files changed, 8 insertions, 12 deletions
diff --git a/OpenSim/Framework/Servers/HttpServer/BaseHttpServer.cs b/OpenSim/Framework/Servers/HttpServer/BaseHttpServer.cs
index fd0621b..fce3671 100644
--- a/OpenSim/Framework/Servers/HttpServer/BaseHttpServer.cs
+++ b/OpenSim/Framework/Servers/HttpServer/BaseHttpServer.cs
@@ -56,7 +56,6 @@ namespace OpenSim.Framework.Servers.HttpServer
56 private volatile int NotSocketErrors = 0; 56 private volatile int NotSocketErrors = 0;
57 public volatile bool HTTPDRunning = false; 57 public volatile bool HTTPDRunning = false;
58 58
59 protected Thread m_workerThread;
60 // protected HttpListener m_httpListener; 59 // protected HttpListener m_httpListener;
61 protected CoolHTTPListener m_httpListener2; 60 protected CoolHTTPListener m_httpListener2;
62 protected Dictionary<string, XmlRpcMethod> m_rpcHandlers = new Dictionary<string, XmlRpcMethod>(); 61 protected Dictionary<string, XmlRpcMethod> m_rpcHandlers = new Dictionary<string, XmlRpcMethod>();
@@ -243,7 +242,8 @@ namespace OpenSim.Framework.Servers.HttpServer
243 242
244 public List<string> GetPollServiceHandlerKeys() 243 public List<string> GetPollServiceHandlerKeys()
245 { 244 {
246 return new List<string>(m_pollHandlers.Keys); 245 lock (m_pollHandlers)
246 return new List<string>(m_pollHandlers.Keys);
247 } 247 }
248 248
249 // Note that the agent string is provided simply to differentiate 249 // Note that the agent string is provided simply to differentiate
@@ -1824,20 +1824,16 @@ namespace OpenSim.Framework.Servers.HttpServer
1824 1824
1825 public bool RemoveAgentHandler(string agent, IHttpAgentHandler handler) 1825 public bool RemoveAgentHandler(string agent, IHttpAgentHandler handler)
1826 { 1826 {
1827 try 1827 lock (m_agentHandlers)
1828 { 1828 {
1829 lock (m_agentHandlers) 1829 IHttpAgentHandler foundHandler;
1830
1831 if (m_agentHandlers.TryGetValue(agent, out foundHandler) && foundHandler == handler)
1830 { 1832 {
1831 if (handler == m_agentHandlers[agent]) 1833 m_agentHandlers.Remove(agent);
1832 { 1834 return true;
1833 m_agentHandlers.Remove(agent);
1834 return true;
1835 }
1836 } 1835 }
1837 } 1836 }
1838 catch(KeyNotFoundException)
1839 {
1840 }
1841 1837
1842 return false; 1838 return false;
1843 } 1839 }