[PATCH 0/1] MR8651: urlmon/tests: Fix out-of-bound write into tmp_dir.

[Yuxuan Shui (＠yshui)]

GetTempPathW length parameter is in number of chars, not number of bytes.

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

[PATCH 1/1] urlmon/tests: Fix out-of-bound write into tmp_dir.

[Yuxuan Shui]

From: Yuxuan Shui <yshui(a)codeweavers.com>

GetTempPathW length parameter is in number of chars, not number of bytes.
---
 dlls/urlmon/tests/sec_mgr.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/dlls/urlmon/tests/sec_mgr.c b/dlls/urlmon/tests/sec_mgr.c
index f3624bac2f0..cf2ab577cd0 100644
--- a/dlls/urlmon/tests/sec_mgr.c
+++ b/dlls/urlmon/tests/sec_mgr.c
@@ -2201,7 +2201,7 @@ static void test_IPersistFile_iface(IPersistFile *persist_file_save, IZoneIdenti
     ok(hres == S_OK,
         "Unexpected Remove result: 0x%08lx, expected result: 0x%08lx\n", hres, S_OK);
 
-    ok(GetTempPathW(sizeof(tmp_dir), tmp_dir),
+    ok(GetTempPathW(ARRAY_SIZE(tmp_dir), tmp_dir),
         "GetTempPathW failed with: 0x%08lx\n", GetLastError());
 
     ok(lstrcatW(tmp_dir, sub_dir) == tmp_dir, "lstrcatW failed\n");
-- 
GitLab

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

[PATCH 0/1] MR8651: urlmon/tests: Fix out-of-bound write into tmp_dir. - approved

[Jacek Caban (＠jacek)]

This merge request was approved by Jacek Caban.
-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/8651