diff options
author | Justin Clark-Casey (justincc) | 2011-09-06 22:48:05 +0100 |
---|---|---|
committer | Justin Clark-Casey (justincc) | 2011-09-06 22:48:05 +0100 |
commit | e6cd4defdbee5d6df9cb5809ab66619ac1e51c9b (patch) | |
tree | b2ac8bb3df9d27c0ec29b77f8555537811dae5d3 /OpenSim/Region/Framework/Scenes | |
parent | Comment out SceneViewer.Reset() and stop calling from Close() since this is u... (diff) | |
download | opensim-SC-e6cd4defdbee5d6df9cb5809ab66619ac1e51c9b.zip opensim-SC-e6cd4defdbee5d6df9cb5809ab66619ac1e51c9b.tar.gz opensim-SC-e6cd4defdbee5d6df9cb5809ab66619ac1e51c9b.tar.bz2 opensim-SC-e6cd4defdbee5d6df9cb5809ab66619ac1e51c9b.tar.xz |
Lock m_pendingObjects when calling GetPendingObjectsCount().
This is only called by a region console command.
We should also be locking m_partsUpdateQueue when dequeueing the next part, or locking m_pendingObjects in QueuePartForUpdate().
However, I won't do this now since I don't have time to analyze how this would affect liveness.
Diffstat (limited to 'OpenSim/Region/Framework/Scenes')
-rw-r--r-- | OpenSim/Region/Framework/Scenes/SceneViewer.cs | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/OpenSim/Region/Framework/Scenes/SceneViewer.cs b/OpenSim/Region/Framework/Scenes/SceneViewer.cs index 9ad80b8..50e1e39 100644 --- a/OpenSim/Region/Framework/Scenes/SceneViewer.cs +++ b/OpenSim/Region/Framework/Scenes/SceneViewer.cs | |||
@@ -231,7 +231,8 @@ namespace OpenSim.Region.Framework.Scenes | |||
231 | public int GetPendingObjectsCount() | 231 | public int GetPendingObjectsCount() |
232 | { | 232 | { |
233 | if (m_pendingObjects != null) | 233 | if (m_pendingObjects != null) |
234 | return m_pendingObjects.Count; | 234 | lock (m_pendingObjects) |
235 | return m_pendingObjects.Count; | ||
235 | 236 | ||
236 | return 0; | 237 | return 0; |
237 | } | 238 | } |