From db1ed17ac46bc721e89990cd015fc3bcde956de2 Mon Sep 17 00:00:00 2001 From: Andreas Gottschling Date: Fri, 27 Sep 2019 12:02:06 -0400 Subject: [PATCH] drisw: Fix shared memory leak on drawable resize XDestroyImage will mark the segment as to-be-destroyed, but it will persist until we detach it, and we weren't doing so. Cc: mesa-stable@lists.freedesktop.org Gitlab: https://gitlab.freedesktop.org/mesa/mesa/issues/121 Reviewed-by: Adam Jackson (cherry picked from commit c5a2ccec5e9fa60bbec5e7b574877f8af4c14462) --- src/glx/drisw_glx.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/src/glx/drisw_glx.c b/src/glx/drisw_glx.c index f38dbbca2df..39b484b2daa 100644 --- a/src/glx/drisw_glx.c +++ b/src/glx/drisw_glx.c @@ -86,6 +86,8 @@ XCreateDrawable(struct drisw_drawable * pdp, int shmid, Display * dpy) if (pdp->ximage) { XDestroyImage(pdp->ximage); pdp->ximage = NULL; + if ((pdp->shminfo.shmid > 0) && (shmid != pdp->shminfo.shmid)) + XShmDetach(dpy, &pdp->shminfo); } if (!xshm_error && shmid >= 0) {