Add missing lock to ensure thread-safe mutations of _activeInnerStreams in VirtualConnectionClient.cs

SteveSandersonMS · SteveSandersonMS · commit 3fc19ea77bdb · 2016-06-14T10:16:40.000+01:00
diff --git a/src/Microsoft.AspNetCore.NodeServices/HostingModels/VirtualConnections/VirtualConnectionClient.cs b/src/Microsoft.AspNetCore.NodeServices/HostingModels/VirtualConnections/VirtualConnectionClient.cs
@@ -46,7 +46,11 @@ public Stream OpenVirtualConnection()
 
             var id = Interlocked.Increment(ref _nextInnerStreamId);
             var newInnerStream = new VirtualConnection(id, this);
-            _activeInnerStreams.Add(id, newInnerStream);
+            lock (_activeInnerStreams)
+            {
+                _activeInnerStreams.Add(id, newInnerStream);
+            }
+
             return newInnerStream;
         }
 
