winex11: Free correct pointer when using xf86vm.

After allocation the pointer gets shifted by sizeof(xf86vm_modes), so we have to undo that before calling free. Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=55909

winex11: Free correct pointer when using xf86vm.
18522a05 · Fabian Maurer · Alexandre Julliard · f341a693 · 18522a05
Commit 18522a05 authored Nov 19, 2023 by Fabian Maurer Committed by Alexandre Julliard Nov 21, 2023
Hide whitespace changes
Inline Side-by-side

Showing with 3 additions and 2 deletions

xvidmode.c dlls/winex11.drv/xvidmode.c +3 -2

No files found.
--- a/dlls/winex11.drv/xvidmode.c
+++ b/dlls/winex11.drv/xvidmode.c
@@ -159,11 +159,12 @@ static void xf86vm_free_modes(DEVMODEW *modes)

    if (modes)
    {
+        BYTE *ptr = (BYTE *)modes - sizeof(xf86vm_modes);
        assert(modes[0].dmDriverExtra == sizeof(XF86VidModeModeInfo *));
-        memcpy(&xf86vm_modes, (BYTE *)modes - sizeof(xf86vm_modes), sizeof(xf86vm_modes));
+        memcpy(&xf86vm_modes, ptr, sizeof(xf86vm_modes));
        XFree(xf86vm_modes);
+        free(ptr);
    }
-    free(modes);
 }

 static BOOL xf86vm_get_current_mode(x11drv_settings_id id, DEVMODEW *mode)