Don't pulse without a lock

mythz · mythz · commit 6e105a10ea51 · 2013-06-13T02:24:50.000-04:00
diff --git a/src/ServiceStack.Redis/PooledRedisClientManager.cs b/src/ServiceStack.Redis/PooledRedisClientManager.cs
@@ -385,8 +385,10 @@ public void DisposeClient(RedisNativeClient client)
             }
 
             //Client not found in any pool, pulse both pools.
-            Monitor.PulseAll(readClients);
-            Monitor.PulseAll(writeClients);
+            lock (readClients)
+                Monitor.PulseAll(readClients);
+            lock (writeClients)
+                Monitor.PulseAll(writeClients);
         }
 
         /// <summary>

Original file line number	Diff line number	Diff line change
`@@ -385,8 +385,10 @@ public void DisposeClient(RedisNativeClient client)`
`385`	`385`	`}`
`386`	`386`
`387`	`387`	`//Client not found in any pool, pulse both pools.`
`388`		`- Monitor.PulseAll(readClients);`
`389`		`- Monitor.PulseAll(writeClients);`
	`388`	`+ lock (readClients)`
	`389`	`+ Monitor.PulseAll(readClients);`
	`390`	`+ lock (writeClients)`
	`391`	`+ Monitor.PulseAll(writeClients);`
`390`	`392`	`}`
`391`	`393`
`392`	`394`	`/// <summary>`