On Tue, Mar 29, 2011 at 06:08:15PM +0300, Tiago Vignatti wrote:
> Signed-off-by: Tiago Vignatti <[email protected]>
> ---
>  Xi/getselev.c |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
> 
> diff --git a/Xi/getselev.c b/Xi/getselev.c
> index 09a53f4..7304738 100644
> --- a/Xi/getselev.c
> +++ b/Xi/getselev.c
> @@ -152,8 +152,8 @@ ProcXGetSelectedExtensionEvents(ClientPtr client)
>      if (total_length) {
>       client->pSwapReplyFunc = (ReplySwapPtr) Swap32Write;
>       WriteSwappedDataToClient(client, total_length, buf);
> -     free(buf);
>      }
> +    free(buf);
>      return Success;
>  }
>  
don't see why this would be a leak, given that it only allocates a buffer
for a nonzero length. either way, doesn't hurt, so 

Reviewed-by: Peter Hutterer <[email protected]>

Cheers,
  Peter
_______________________________________________
[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