Misha Koshelev : wininet/tests: Add test to show that InternetCloseHandle closes open child handles too.

Alexandre Julliard julliard at winehq.org
Fri Sep 21 07:23:02 CDT 2007


Module: wine
Branch: master
Commit: 01de104a75737887837154712c309860bcbc7a52
URL:    http://source.winehq.org/git/wine.git/?a=commit;h=01de104a75737887837154712c309860bcbc7a52

Author: Misha Koshelev <mk144210 at bcm.edu>
Date:   Thu Sep 20 21:00:38 2007 -0500

wininet/tests: Add test to show that InternetCloseHandle closes open child handles too.

---

 dlls/wininet/tests/http.c |    8 ++++----
 1 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/dlls/wininet/tests/http.c b/dlls/wininet/tests/http.c
index 1040689..589fe8a 100644
--- a/dlls/wininet/tests/http.c
+++ b/dlls/wininet/tests/http.c
@@ -447,10 +447,10 @@ abort:
             "Double close of handle should have set ERROR_INVALID_HANDLE instead of %u\n",
             GetLastError());
     }
-    if (hic != 0x0) {
-        rc = InternetCloseHandle(hic);
-        ok ((rc != 0), "InternetCloseHandle of handle opened by InternetConnectA failed\n");
-    }
+    /* We intentionally do not close the handle opened by InternetConnectA as this
+     * tickles bug #9479: native closes child internet handles when the parent handles
+     * are closed. This is verified below by checking that the number of
+     * INTERNET_STATUS_HANDLE_CLOSING notifications matches the number expected. */
     if (hi != 0x0) {
       SET_WINE_ALLOW(INTERNET_STATUS_HANDLE_CLOSING);
       rc = InternetCloseHandle(hi);




More information about the wine-cvs mailing list