Merge pull request #35 from atc-net/hotfix/Fix-syncLock-dispose

fix: Move the syncLock.Dispose to the right place
atc-net · Feb 14, 2024 · 4fc749d · 4fc749d
2 parents a85f89b + 1f8459e
commit 4fc749d
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/src/Atc.Network/Tcp/TcpClient.cs b/src/Atc.Network/Tcp/TcpClient.cs
@@ -360,6 +360,8 @@ protected virtual void Dispose(
 
         DisposeCancellationTokenAndTask();
         DisposeTcpClientAndStream();
+
+        syncLock.Dispose();
     }
 
     private void InvokeConnected()
@@ -822,8 +824,6 @@ private void DisposeTcpClientAndStream()
             tcpClient.Dispose();
             tcpClient = null;
         }
-
-        syncLock.Dispose();
     }
 
     private void CancellationTokenCallback()

diff --git a/src/Atc.Network/Udp/UdpClient.cs b/src/Atc.Network/Udp/UdpClient.cs
@@ -256,6 +256,8 @@ protected virtual void Dispose(
 
         DisposeCancellationTokenAndTask();
         DisposeSocket();
+
+        syncLock.Dispose();
     }
 
     private void InvokeConnected()
@@ -548,7 +550,5 @@ private void DisposeSocket()
             socket.Dispose();
             socket = null;
         }
-
-        syncLock.Dispose();
     }
 }