[PATCH 0/1] MR8650: urlmon/tests: fixed argument size passed to GetTempPathW

[Marcus Meissner (＠msmeissn)]

Signed-off-by: Marcus Meissner <marcus(a)jet.franken.de>

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

[PATCH 1/1] urlmon/tests: fixed argument size passed to GetTempPathW

[Marcus Meissner]

From: Marcus Meissner <marcus(a)jet.franken.de>

Signed-off-by: Marcus Meissner <marcus(a)jet.franken.de>
---
 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..9b831c87106 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(sizeof(tmp_dir), ARRAY_SIZE(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/8650

[Dmitry Timoshkov (＠dmitry)]

Dmitry Timoshkov (@dmitry) commented about dlls/urlmon/tests/sec_mgr.c:
> 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(sizeof(tmp_dir), ARRAY_SIZE(tmp_dir)),

Shouldn't this be 'GetTempPathW(ARRAY_SIZE(tmp_dir), tmp_dir);'?

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/8650#note_111134