[PATCH v2 10/25] [libx11] Fixed memory leak by adding Xfree

Erkki Seppälä erkki.seppala at vincit.fi
Mon Jan 31 04:02:02 PST 2011


Variable "colormap_ret" goes out of scope

Reviewed-by: Ander Conselvan de Oliveira <ander.conselvan-de-oliveira at nokia.com>
Signed-off-by: Erkki Seppälä <erkki.seppala at vincit.fi>
---
 modules/im/ximcp/imRm.c |    2 ++
 1 files changed, 2 insertions(+), 0 deletions(-)

diff --git a/modules/im/ximcp/imRm.c b/modules/im/ximcp/imRm.c
index fa86979..3d09b81 100644
--- a/modules/im/ximcp/imRm.c
+++ b/modules/im/ximcp/imRm.c
@@ -2821,6 +2821,8 @@ _XimEncodeLocalPreeditValue(
 				ic->core.focus_window, &colormap_ret,
 				&count, (Atom)p->value)))
 	    return False;
+
+	Xfree(colormap_ret);
     }
     return True;
 }
-- 
1.7.0.4



More information about the xorg-devel mailing list