Always decrement count on HTTP/2 stream completion (#3087)

halter73 · web-flow · commit 5db6394769e2 · 2018-11-09T12:40:51.000-08:00
#3079
diff --git a/src/Kestrel.Core/Internal/Http2/Http2Connection.cs b/src/Kestrel.Core/Internal/Http2/Http2Connection.cs
@@ -1045,11 +1045,11 @@ void IHttp2StreamLifetimeHandler.OnStreamCompleted(int streamId)
         {
             lock (_stateLock)
             {
+                _activeStreamCount--;
+
                 // Get, Add, Remove so the steam is always registered in at least one collection at a time.
                 if (_streams.TryGetValue(streamId, out var stream))
                 {
-                    _activeStreamCount--;
-
                     if (stream.IsDraining)
                     {
                         stream.DrainExpiration =

Original file line number	Diff line number	Diff line change
`@@ -1045,11 +1045,11 @@ void IHttp2StreamLifetimeHandler.OnStreamCompleted(int streamId)`
`1045`	`1045`	`{`
`1046`	`1046`	`lock (_stateLock)`
`1047`	`1047`	`{`
	`1048`	`+ _activeStreamCount--;`
	`1049`	`+`
`1048`	`1050`	`// Get, Add, Remove so the steam is always registered in at least one collection at a time.`
`1049`	`1051`	`if (_streams.TryGetValue(streamId, out var stream))`
`1050`	`1052`	`{`
`1051`		`- _activeStreamCount--;`
`1052`		`-`
`1053`	`1053`	`if (stream.IsDraining)`
`1054`	`1054`	`{`
`1055`	`1055`	`stream.DrainExpiration =`