[PATCH resend 1/2] wininet/tests: Test adding HTTP header with empty value.
Ziqing Hui
zhui at codeweavers.com
Sun Nov 21 20:25:14 CST 2021
Signed-off-by: Ziqing Hui <zhui at codeweavers.com>
---
Resend: Please take a look at this patch set. Don't ignore it for the test failure.
I think the test failure is not caused by this patch. See HTTP tests with a empty patch:
https://testbot.winehq.org/JobDetails.pl?Key=101700
https://testbot.winehq.org/JobDetails.pl?Key=101701
dlls/wininet/tests/http.c | 26 ++++++++++++++++++++++++++
1 file changed, 26 insertions(+)
diff --git a/dlls/wininet/tests/http.c b/dlls/wininet/tests/http.c
index 3b8f16a9fb2..c3f33b4e0b7 100644
--- a/dlls/wininet/tests/http.c
+++ b/dlls/wininet/tests/http.c
@@ -2063,6 +2063,32 @@ static void HttpHeaders_test(void)
"header still present\n");
ok(GetLastError() == ERROR_HTTP_HEADER_NOT_FOUND, "got %u\n", GetLastError());
+ /* Adding header with empty value should cause a failure */
+ todo_wine
+ {
+ SetLastError(0xdeadbeef);
+ ok(!HttpAddRequestHeadersA(hRequest, "EmptyTest1:", -1, HTTP_ADDREQ_FLAG_ADD), "Empty header should not be added.\n");
+ ok(GetLastError() == ERROR_INVALID_PARAMETER, "Got unexpected error code %u.\n", GetLastError());
+
+ SetLastError(0xdeadbeef);
+ ok(!HttpAddRequestHeadersA(hRequest, "EmptyTest2:\r\n", -1, HTTP_ADDREQ_FLAG_ADD), "Empty header should not be added.\n");
+ ok(GetLastError() == ERROR_INVALID_PARAMETER, "Got unexpected error code %u.\n", GetLastError());
+
+ len = sizeof(buffer);
+ strcpy(buffer, "EmptyTest1");
+ SetLastError(0xdeadbeef);
+ ok(!HttpQueryInfoA(hRequest, HTTP_QUERY_CUSTOM | HTTP_QUERY_FLAG_REQUEST_HEADERS, buffer, &len, NULL),
+ "Header with empty value is present.\n");
+ ok(GetLastError() == ERROR_HTTP_HEADER_NOT_FOUND, "Got unexpected error code %u.\n", GetLastError());
+
+ len = sizeof(buffer);
+ strcpy(buffer, "EmptyTest2");
+ SetLastError(0xdeadbeef);
+ ok(!HttpQueryInfoA(hRequest, HTTP_QUERY_CUSTOM | HTTP_QUERY_FLAG_REQUEST_HEADERS, buffer, &len, NULL),
+ "Header with empty value is present.\n");
+ ok(GetLastError() == ERROR_HTTP_HEADER_NOT_FOUND, "Got unexpected error code %u.\n", GetLastError());
+ }
+
ok(InternetCloseHandle(hRequest), "Close request handle failed\n");
done:
ok(InternetCloseHandle(hConnect), "Close connect handle failed\n");
--
2.25.1
More information about the wine-devel
mailing list