Fix regression in CancelAsyncConnections test using Native SNI - repr…

…oduced locally (#1226)
dotnet · Aug 25, 2021 · 2b74ca1 · 2b74ca1
1 parent 6dde935
commit 2b74ca1
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/src/Microsoft.Data.SqlClient/netcore/src/Microsoft/Data/SqlClient/TdsParserStateObject.cs b/src/Microsoft.Data.SqlClient/netcore/src/Microsoft/Data/SqlClient/TdsParserStateObject.cs
@@ -2527,7 +2527,8 @@ internal void ReadSni(TaskCompletionSource<object> completion)
                     // Evaluate this condition for MANAGED_SNI. This may not be needed because the network call is happening Async and only the callback can receive a success.
                     ReadAsyncCallback(IntPtr.Zero, readPacket, 0);
 
-                    if (!IsPacketEmpty(readPacket))
+                    // Only release packet for Managed SNI as for Native SNI packet is released in finally block.
+                    if (TdsParserStateObjectFactory.UseManagedSNI && !IsPacketEmpty(readPacket))
                     {
                         ReleasePacket(readPacket);
                     }