On 01/31/11 04:02 AM, Erkki Seppälä wrote:
> Variable "colormap_ret" goes out of scope
> 
> Reviewed-by: Ander Conselvan de Oliveira 
> <[email protected]>
> Signed-off-by: Erkki Seppälä <[email protected]>
> ---
>  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 2e2c31f..fa86979 100644
> --- a/modules/im/ximcp/imRm.c
> +++ b/modules/im/ximcp/imRm.c
> @@ -2841,6 +2841,8 @@ _XimEncodeLocalStatusValue(
>                               ic->core.focus_window, &colormap_ret,
>                               &count, (Atom)p->value)))
>           return False;
> +
> +     Xfree(colormap_ret);
>      }
>      return True;
>  }

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