[PATCH 0/1] MR7278: winhttp: Check for the end of the text when stripping trailing newlines from headers (ASan).
Signed-off-by: Nikolay Sivov <nsivov(a)codeweavers.com> -- https://gitlab.winehq.org/wine/wine/-/merge_requests/7278
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
This merge request was approved by Hans Leidekker. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/7278
participants (3)
-
Hans Leidekker (@hans) -
Nikolay Sivov -
Nikolay Sivov (@nsivov)