winhttp/tests: Fix resource leak.

Signed-off-by: Andrey Gusev <andrey.goosev@gmail.com> Signed-off-by: Hans Leidekker <hans@codeweavers.com> Signed-off-by: Alexandre Julliard <julliard@winehq.org>

winhttp/tests: Fix resource leak.
67e80856 · Andrey Gusev · Alexandre Julliard · 53b73055 · 67e80856 · 67e80856
Commit 67e80856 authored Jan 05, 2018 by Andrey Gusev Committed by Alexandre Julliard Jan 05, 2018
Hide whitespace changes
Inline Side-by-side

Showing with 7 additions and 0 deletions

notification.c dlls/winhttp/tests/notification.c +3 -0

winhttp.c dlls/winhttp/tests/winhttp.c +4 -0

No files found.
--- a/dlls/winhttp/tests/notification.c
+++ b/dlls/winhttp/tests/notification.c
@@ -1033,7 +1033,10 @@ START_TEST (notification)
    ret = WaitForSingleObject( si.event, 10000 );
    ok(ret == WAIT_OBJECT_0, "failed to start winhttp test server %u\n", GetLastError());
    if (ret != WAIT_OBJECT_0)
+    {
+        CloseHandle(thread);
        return;
+    }

    test_persistent_connection( si.port );


--- a/dlls/winhttp/tests/winhttp.c
+++ b/dlls/winhttp/tests/winhttp.c
@@ -4475,7 +4475,10 @@ START_TEST (winhttp)
    ret = WaitForSingleObject(si.event, 10000);
    ok(ret == WAIT_OBJECT_0, "failed to start winhttp test server %u\n", GetLastError());
    if (ret != WAIT_OBJECT_0)
+    {
+        CloseHandle(thread);
        return;
+    }

    test_IWinHttpRequest(si.port);
    test_connection_info(si.port);
@@ -4495,4 +4498,5 @@ START_TEST (winhttp)
    test_basic_request(si.port, NULL, quitW);

    WaitForSingleObject(thread, 3000);
+    CloseHandle(thread);
 }