Move SenseRepeaters.Count check inside the SenseRepeatListLock.

No methods in the List class are thread safe in the MS specification/documentation
2012-03-02 00:28:37 +00:00 · 2012-03-02 00:28:37 +00:00 · 54d0514b13
parent 58b1c3cec0
commit 54d0514b13
1 changed files with 4 additions and 4 deletions
--- a/OpenSim/Region/ScriptEngine/Shared/Api/Implementation/Plugins/SensorRepeat.cs
+++ b/OpenSim/Region/ScriptEngine/Shared/Api/Implementation/Plugins/SensorRepeat.cs
@ -157,12 +157,12 @@ namespace OpenSim.Region.ScriptEngine.Shared.Api.Plugins

        public void CheckSenseRepeaterEvents()
        {
-            // Nothing to do here?
-            if (SenseRepeaters.Count == 0)
-                return;
-
            lock (SenseRepeatListLock)
            {
+                // Nothing to do here?
+                if (SenseRepeaters.Count == 0)
+                    return;
+
                // Go through all timers
                foreach (SenseRepeatClass ts in SenseRepeaters)
                {