[PATCH 0/1] MR9205: winex11: Avoid freeing default colormap.

[Rémi Bernon]

There's some weird XFreeColorMap error happening in d3d8 tests, I'm not able to reproduce it but it could be this. Anyway, it's incorrect to free default_colormap, so lets avoid doing it regardless.

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

[PATCH 1/1] winex11: Avoid freeing default colormap.

[Rémi Bernon]

From: Rémi Bernon <rbernon(a)codeweavers.com>

---
 dlls/winex11.drv/init.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/dlls/winex11.drv/init.c b/dlls/winex11.drv/init.c
index 08810b44024..1211cbabaa7 100644
--- a/dlls/winex11.drv/init.c
+++ b/dlls/winex11.drv/init.c
@@ -456,7 +456,7 @@ Window x11drv_client_surface_create( HWND hwnd, int format, struct client_surfac
 
 failed:
     if (surface) client_surface_release( &surface->client );
-    XFreeColormap( gdi_display, colormap );
+    if (colormap != default_colormap) XFreeColormap( gdi_display, colormap );
     return None;
 }
 
-- 
GitLab

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