[PATCH xserver 4/9] dmx/glxProxy: don't free the glx pixmap twice if AddResource fails

Julien Cristau jcristau at debian.org
Mon Mar 7 22:20:29 UTC 2016


Signed-off-by: Julien Cristau <jcristau at debian.org>
---
 hw/dmx/glxProxy/glxcmds.c | 5 +----
 1 file changed, 1 insertion(+), 4 deletions(-)

diff --git a/hw/dmx/glxProxy/glxcmds.c b/hw/dmx/glxProxy/glxcmds.c
index ddcb981..a77d556 100644
--- a/hw/dmx/glxProxy/glxcmds.c
+++ b/hw/dmx/glxProxy/glxcmds.c
@@ -2010,11 +2010,8 @@ CreateGLXPixmap(__GLXclientState * cl,
         XFlush(dpy);
     }
 
-    if (!(AddResource(glxpixmapId, __glXPixmapRes, pGlxPixmap))) {
-        free(pGlxPixmap->be_xids);
-        free(pGlxPixmap);
+    if (!(AddResource(glxpixmapId, __glXPixmapRes, pGlxPixmap)))
         return BadAlloc;
-    }
 
     return Success;
 }
-- 
2.7.0



More information about the xorg-devel mailing list