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

Alan Coopersmith alan.coopersmith at oracle.com
Mon Jan 31 23:19:56 PST 2011


On 01/31/11 04:02 AM, Erkki Seppälä wrote:
> Variable "colormap_ret" goes out of scope
> 
> Signed-off-by: Erkki Seppälä <erkki.seppala at vincit.fi>
> ---
>  modules/im/ximcp/imRmAttr.c |    1 +
>  1 files changed, 1 insertions(+), 0 deletions(-)
> 
> diff --git a/modules/im/ximcp/imRmAttr.c b/modules/im/ximcp/imRmAttr.c
> index ae053c9..5035df5 100644
> --- a/modules/im/ximcp/imRmAttr.c
> +++ b/modules/im/ximcp/imRmAttr.c
> @@ -957,6 +957,7 @@ _XimEncodePreeditValue(
>  				&count, (Atom)p->value)))
>  	    return False;
>  
> +	XFree(colormap_ret);
>      } else if (res->xrm_name == XrmStringToQuark(XNFontSet)) {
>  	int		  list_ret;
>  	XFontStruct	**struct_list;

Reviewed-by: Alan Coopersmith <alan.coopersmith at oracle.com>

-- 
	-Alan Coopersmith-        alan.coopersmith at oracle.com
	 Oracle Solaris Platform Engineering: X Window System



More information about the xorg-devel mailing list