[PATCH 3/5] dlls/kernel32/test: fully allocate KEY_VALUE_*_INFORMATION

Eric Pouech eric.pouech at gmail.com
Thu Feb 17 04:16:36 CST 2022


this prevents mingw-gcc version 11 to generate warnings (-Warray-bounds)

Signed-off-by: Eric Pouech <eric.pouech at gmail.com>

---
 dlls/ntdll/tests/reg.c |    6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/dlls/ntdll/tests/reg.c b/dlls/ntdll/tests/reg.c
index bd8d531edf0..7d069666604 100644
--- a/dlls/ntdll/tests/reg.c
+++ b/dlls/ntdll/tests/reg.c
@@ -742,7 +742,7 @@ static void test_NtQueryValueKey(void)
     ok(status == STATUS_SUCCESS, "NtOpenKey Failed: 0x%08x\n", status);
 
     len = FIELD_OFFSET(KEY_VALUE_BASIC_INFORMATION, Name[0]);
-    basic_info = HeapAlloc(GetProcessHeap(), 0, len);
+    basic_info = HeapAlloc(GetProcessHeap(), 0, sizeof(*basic_info));
     status = pNtQueryValueKey(key, &ValName, KeyValueBasicInformation, basic_info, len, &len);
     ok(status == STATUS_BUFFER_OVERFLOW, "NtQueryValueKey should have returned STATUS_BUFFER_OVERFLOW instead of 0x%08x\n", status);
     ok(basic_info->TitleIndex == 0, "NtQueryValueKey returned wrong TitleIndex %d\n", basic_info->TitleIndex);
@@ -761,7 +761,7 @@ static void test_NtQueryValueKey(void)
     HeapFree(GetProcessHeap(), 0, basic_info);
 
     len = FIELD_OFFSET(KEY_VALUE_PARTIAL_INFORMATION, Data[0]);
-    partial_info = HeapAlloc(GetProcessHeap(), 0, len);
+    partial_info = HeapAlloc(GetProcessHeap(), 0, sizeof(*partial_info));
     status = pNtQueryValueKey(key, &ValName, KeyValuePartialInformation, partial_info, len, &len);
     ok(status == STATUS_BUFFER_OVERFLOW, "NtQueryValueKey should have returned STATUS_BUFFER_OVERFLOW instead of 0x%08x\n", status);
     ok(partial_info->TitleIndex == 0, "NtQueryValueKey returned wrong TitleIndex %d\n", partial_info->TitleIndex);
@@ -780,7 +780,7 @@ static void test_NtQueryValueKey(void)
     HeapFree(GetProcessHeap(), 0, partial_info);
 
     len = FIELD_OFFSET(KEY_VALUE_FULL_INFORMATION, Name[0]);
-    full_info = HeapAlloc(GetProcessHeap(), 0, len);
+    full_info = HeapAlloc(GetProcessHeap(), 0, sizeof(*full_info));
     status = pNtQueryValueKey(key, &ValName, KeyValueFullInformation, full_info, len, &len);
     ok(status == STATUS_BUFFER_OVERFLOW, "NtQueryValueKey should have returned STATUS_BUFFER_OVERFLOW instead of 0x%08x\n", status);
     ok(full_info->TitleIndex == 0, "NtQueryValueKey returned wrong TitleIndex %d\n", full_info->TitleIndex);




More information about the wine-devel mailing list