From d2e289c70f2b9fb882f5b3992b610a9ea5c8357c Mon Sep 17 00:00:00 2001
From: Brian Paul <brian.paul@tungstengraphics.com>
Date: Tue, 9 Aug 2005 15:17:19 +0000
Subject: actually free the renderbuffer in xmesa_delete_renderbuffer(), fixes
 a memory leak

---
 src/mesa/drivers/x11/xm_buffer.c | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

(limited to 'src/mesa/drivers/x11')

diff --git a/src/mesa/drivers/x11/xm_buffer.c b/src/mesa/drivers/x11/xm_buffer.c
index 9f481aaba1..593ac2bb19 100644
--- a/src/mesa/drivers/x11/xm_buffer.c
+++ b/src/mesa/drivers/x11/xm_buffer.c
@@ -33,7 +33,11 @@
 static void
 xmesa_delete_renderbuffer(struct gl_renderbuffer *rb)
 {
-   /* XXX this routine should really delete the attached ximage, etc. */
+   /* XXX Note: the ximage or Pixmap attached to this renderbuffer
+    * should probably get freed here, but that's currently done in
+    * XMesaDestroyBuffer().
+    */
+   _mesa_free(rb);
 }
 
 
-- 
cgit v1.2.3