Hans Leidekker : wininet: Calculate the header size if needed in HttpSendRequestW.

Alexandre Julliard julliard at winehq.org
Fri Jan 16 07:41:52 CST 2009


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

Author: Hans Leidekker <hans at codeweavers.com>
Date:   Thu Jan 15 16:41:53 2009 +0100

wininet: Calculate the header size if needed in HttpSendRequestW.

---

 dlls/wininet/http.c       |    9 +++++++--
 dlls/wininet/tests/http.c |   28 ++++++++++++++++++++++++++++
 2 files changed, 35 insertions(+), 2 deletions(-)

diff --git a/dlls/wininet/http.c b/dlls/wininet/http.c
index ea4e456..0dd68fa 100644
--- a/dlls/wininet/http.c
+++ b/dlls/wininet/http.c
@@ -2897,8 +2897,13 @@ BOOL WINAPI HttpSendRequestW(HINTERNET hHttpRequest, LPCWSTR lpszHeaders,
         req = &workRequest.u.HttpSendRequestW;
         if (lpszHeaders)
         {
-            req->lpszHeader = HeapAlloc(GetProcessHeap(), 0, dwHeaderLength * sizeof(WCHAR));
-            memcpy(req->lpszHeader, lpszHeaders, dwHeaderLength * sizeof(WCHAR));
+            DWORD size;
+
+            if (dwHeaderLength == ~0u) size = (strlenW(lpszHeaders) + 1) * sizeof(WCHAR);
+            else size = dwHeaderLength * sizeof(WCHAR);
+
+            req->lpszHeader = HeapAlloc(GetProcessHeap(), 0, size);
+            memcpy(req->lpszHeader, lpszHeaders, size);
         }
         else
             req->lpszHeader = 0;
diff --git a/dlls/wininet/tests/http.c b/dlls/wininet/tests/http.c
index 19eb982..c5321b4 100644
--- a/dlls/wininet/tests/http.c
+++ b/dlls/wininet/tests/http.c
@@ -1814,6 +1814,33 @@ static void test_http1_1(int port)
     InternetCloseHandle(ses);
 }
 
+static void test_HttpSendRequestW(int port)
+{
+    static const WCHAR header[] = {'U','A','-','C','P','U',':',' ','x','8','6',0};
+    HINTERNET ses, con, req;
+    DWORD error;
+    BOOL ret;
+
+    ses = InternetOpen("winetest", INTERNET_OPEN_TYPE_DIRECT, NULL, NULL, INTERNET_FLAG_ASYNC);
+    ok(ses != NULL, "InternetOpen failed\n");
+
+    con = InternetConnect(ses, "localhost", port, NULL, NULL, INTERNET_SERVICE_HTTP, 0, 0);
+    ok(con != NULL, "InternetConnect failed\n");
+
+    req = HttpOpenRequest(con, NULL, "/test1", NULL, NULL, NULL, 0, 0);
+    ok(req != NULL, "HttpOpenRequest failed\n");
+
+    SetLastError(0xdeadbeef);
+    ret = HttpSendRequestW(req, header, ~0u, NULL, 0);
+    error = GetLastError();
+    ok(!ret, "HttpSendRequestW succeeded\n");
+    ok(error == ERROR_IO_PENDING, "got %u expected ERROR_IO_PENDING\n", error);
+
+    InternetCloseHandle(req);
+    InternetCloseHandle(con);
+    InternetCloseHandle(ses);
+}
+
 static void test_cookie_header(int port)
 {
     HINTERNET ses, con, req;
@@ -2023,6 +2050,7 @@ static void test_http_connection(void)
     test_cookie_header(si.port);
     test_basic_authentication(si.port);
     test_HttpQueryInfo(si.port);
+    test_HttpSendRequestW(si.port);
 
     /* send the basic request again to shutdown the server thread */
     test_basic_request(si.port, "GET", "/quit");




More information about the wine-cvs mailing list