[PATCH] wininet: Free security context in failure case.

[Daniel Lehman]

Signed-off-by: Daniel Lehman <dlehman25(a)gmail.com>
---
 dlls/wininet/netconnection.c | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/dlls/wininet/netconnection.c b/dlls/wininet/netconnection.c
index 4ad22177ee..e3531da17b 100644
--- a/dlls/wininet/netconnection.c
+++ b/dlls/wininet/netconnection.c
@@ -573,6 +573,8 @@ static DWORD netcon_secure_connect_setup(netconn_t *connection, BOOL compat_mode
         WARN("Failed to establish SSL connection: %08x (%u)\n", status, res);
         heap_free(connection->ssl_buf);
         connection->ssl_buf = NULL;
+        if(SecIsValidHandle(&connection->ssl_ctx))
+            DeleteSecurityContext(&connection->ssl_ctx);
         return res ? res : ERROR_INTERNET_SECURITY_CHANNEL_ERROR;
     }
 
-- 
2.17.1

[Marvin]

Hi,

While running your changed tests on Windows, I think I found new failures.
Being a bot and all I'm not very good at pattern recognition, so I might be
wrong, but could you please double-check?

Full results can be found at:
https://testbot.winehq.org/JobDetails.pl?Key=45063

Your paranoid android.


=== debian9 (32 bit report) ===

wininet:
Unhandled exception: page fault on read access to 0x0000003c in 32-bit code (0x7d75e1b0).

Report errors:
wininet:http crashed (c0000005)

=== debian9 (32 bit Chinese:China report) ===

wininet:
Unhandled exception: page fault on read access to 0x0000003c in 32-bit code (0x7d5c31b0).

Report errors:
wininet:http crashed (c0000005)

=== debian9 (32 bit WoW report) ===

wininet:
Unhandled exception: page fault on read access to 0x0000003c in 32-bit code (0xf71eb1b0).

Report errors:
wininet:http crashed (c0000005)

=== debian9 (64 bit WoW report) ===

wininet:
Unhandled exception: page fault on read access to 0x00000068 in 64-bit code (0x00007fda24148ef0).

Report errors:
wininet:http crashed (c0000005)