frederik
/
xserver
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

xwin: fix memleak on freeing pixmaps 

Xwin's DestroyPixmap proc just free()s the PixmapRec directly, w/o catering
for devPrivate's, so leaving a memleak. The correct DIX function for this
is FreePixmap().

Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net>
Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1710>
This commit is contained in:
Enrico Weigelt, metux IT consult 2024-10-03 11:53:06 +02:00
parent dfd1da4bbc
commit eb9aa7a5d2
1 changed files with 1 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
hw/xwin/winmultiwindowwindow.c
View File

@ -1137,9 +1137,7 @@ winDestroyPixmapMultiwindow(PixmapPtr pPixmap)
pPixmapPriv->pbmih = NULL; pPixmapPriv->pbmih = NULL;
/* Free the pixmap memory */ /* Free the pixmap memory */
free(pPixmap); FreePixmap(pPixmap);
pPixmap = NULL;
return TRUE; return TRUE;
} }