user32/tests: Zero initialize DEVMODE before passing it to EnumDisplaySettings().

Zero initialize DEVMODE before passing it to EnumDisplaySettings(), which may write beyond the end of the DEVMODE structure on Windows because the dmDriverExtra field is uninitialized. Signed-off-by: Zhiyi Zhang <zzhang@codeweavers.com> Signed-off-by: Alexandre Julliard <julliard@winehq.org>

user32/tests: Zero initialize DEVMODE before passing it to EnumDisplaySettings().
6d68c38c · Zhiyi Zhang · Alexandre Julliard · ee1880a7 · 6d68c38c
Commit 6d68c38c authored Aug 11, 2020 by Zhiyi Zhang Committed by Alexandre Julliard Aug 11, 2020
Show whitespace changes
Inline Side-by-side

Showing with 1 addition and 0 deletions

monitor.c dlls/user32/tests/monitor.c +1 -0

No files found.
--- a/dlls/user32/tests/monitor.c
+++ b/dlls/user32/tests/monitor.c
@@ -514,6 +514,7 @@ static void test_ChangeDisplaySettingsEx(void)

        devices[device_count].index = device;
        lstrcpyA(devices[device_count].name, dd.DeviceName);
+        memset(&devices[device_count].original_mode, 0, sizeof(devices[device_count].original_mode));
        devices[device_count].original_mode.dmSize = sizeof(devices[device_count].original_mode);
        res = EnumDisplaySettingsA(dd.DeviceName, ENUM_CURRENT_SETTINGS, &devices[device_count].original_mode);
        ok(res, "EnumDisplaySettingsA %s failed, error %#x\n", dd.DeviceName, GetLastError());