Dispose SocketAsyncEventArgs when we dispose the Socket (#2459)

Author: halter73

src/Kestrel.Transport.Sockets/Internal/SocketConnection.cs

@@ -86,6 +86,8 @@ public async Task StartAsync(IConnectionDispatcher connectionDispatcher)
 
                 // Dispose the socket(should noop if already called)
                 _socket.Dispose();
+                _receiver.Dispose();
+                _sender.Dispose();
             }
             catch (Exception ex)
             {

src/Kestrel.Transport.Sockets/Internal/SocketReceiver.cs

@@ -7,7 +7,7 @@
 
 namespace Microsoft.AspNetCore.Server.Kestrel.Transport.Sockets.Internal
 {
-    public class SocketReceiver
+    public class SocketReceiver : IDisposable
     {
         private readonly Socket _socket;
         private readonly SocketAsyncEventArgs _eventArgs = new SocketAsyncEventArgs();
@@ -37,5 +37,10 @@ public SocketAwaitable ReceiveAsync(Memory<byte> buffer)
 
             return _awaitable;
         }
+
+        public void Dispose()
+        {
+            _eventArgs.Dispose();
+        }
     }
 }

src/Kestrel.Transport.Sockets/Internal/SocketSender.cs

@@ -11,7 +11,7 @@
 
 namespace Microsoft.AspNetCore.Server.Kestrel.Transport.Sockets.Internal
 {
-    public class SocketSender
+    public class SocketSender : IDisposable
     {
         private readonly Socket _socket;
         private readonly SocketAsyncEventArgs _eventArgs = new SocketAsyncEventArgs();
@@ -98,5 +98,10 @@ private List<ArraySegment<byte>> GetBufferList(ReadOnlySequence<byte> buffer)
 
             return _bufferList;
         }
+
+        public void Dispose()
+        {
+            _eventArgs.Dispose();
+        }
     }
 }