In TerrainModule, lock m_perClientPatchUpdates when removing entries.

This may have been the trigger CheckSendingPatchesToClients() dictionary out of sync exceptions in today's load test.
Don't need to check ContainsKey() since Remove() returns false on a request to remove a key that it doesn't have

1 files changed, 2 insertions, 3 deletions

diff --git a/OpenSim/Region/CoreModules/World/Terrain/TerrainModule.cs b/OpenSim/Region/CoreModules/World/Terrain/TerrainModule.cs
index a0f9129..cd76693 100644
--- a/OpenSim/Region/CoreModules/World/Terrain/TerrainModule.cs
+++ b/OpenSim/Region/CoreModules/World/Terrain/TerrainModule.cs
@@ -873,10 +873,9 @@ namespace OpenSim.Region.CoreModules.World.Terrain
                 presence.ControllingClient.OnLandUndo -= client_OnLandUndo;
                 presence.ControllingClient.OnUnackedTerrain -= client_OnUnackedTerrain;
             }
-            if (m_perClientPatchUpdates.ContainsKey(client))
-            {
+
+            lock (m_perClientPatchUpdates)
                 m_perClientPatchUpdates.Remove(client);
-            }
         }
         
         /// <summary>