On 01/31/11 04:02 AM, Erkki Seppälä wrote:
> Variable "colormap_ret" goes out of scope
> 
> Signed-off-by: Erkki Seppälä <[email protected]>
> ---
>  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 <[email protected]>

-- 
        -Alan Coopersmith-        [email protected]
         Oracle Solaris Platform Engineering: X Window System

_______________________________________________
[email protected]: X.Org development
Archives: http://lists.x.org/archives/xorg-devel
Info: http://lists.x.org/mailman/listinfo/xorg-devel

Reply via email to