[PATCH 0/1] MR7278: winhttp: Check for the end of the text when stripping trailing newlines from headers (ASan).

[Nikolay Sivov (＠nsivov)]

Signed-off-by: Nikolay Sivov <nsivov(a)codeweavers.com>

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/7278

[PATCH 1/1] winhttp: Check for the end of the text when stripping trailing newlines from headers (ASan).

[Nikolay Sivov]

From: Nikolay Sivov <nsivov(a)codeweavers.com>

Signed-off-by: Nikolay Sivov <nsivov(a)codeweavers.com>
---
 dlls/winhttp/request.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/dlls/winhttp/request.c b/dlls/winhttp/request.c
index d3ec53274cd..26022dba3b1 100644
--- a/dlls/winhttp/request.c
+++ b/dlls/winhttp/request.c
@@ -497,7 +497,7 @@ DWORD add_request_headers( struct request *request, const WCHAR *headers, DWORD
         for (q = p; q < headers + len && *q != '\r' && *q != '\n'; ++q)
             ;
         end = q;
-        while (*q == '\r' || *q == '\n')
+        while (q < headers + len && (*q == '\r' || *q == '\n'))
             ++q;
 
         if ((header = parse_header( p, end - p, FALSE )))
-- 
GitLab

https://gitlab.winehq.org/wine/wine/-/merge_requests/7278

[PATCH 0/1] MR7278: winhttp: Check for the end of the text when stripping trailing newlines from headers (ASan). - approved

[Hans Leidekker (＠hans)]

This merge request was approved by Hans Leidekker.
-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/7278