[Spice-devel] [PATCH] [xf86-video-qxl] Free the region pointer as well; eliminates a memory leak in dfps mode.

[Marc-André Lureau]

ack, thanks

----- Original Message -----
> 
> Signed-off-by: Jeremy White <jwhite at codeweavers.com>
> ---
>  src/dfps.c |    3 +++
>  1 file changed, 3 insertions(+)
> 
> diff --git a/src/dfps.c b/src/dfps.c
> index 5cc1890..4ab20a8 100644
> --- a/src/dfps.c
> +++ b/src/dfps.c
> @@ -169,6 +169,7 @@ static void dfps_solid (PixmapPtr pixmap, int x_1, int
> y_1, int x_2, int y_2)
>      RegionAppend(&info->updated_region, region);
>      RegionValidate(&info->updated_region, &throwaway_bool);
>      RegionUninit(region);
> +    RegionDestroy(region);
>      return;
>  }
>  
> @@ -229,6 +230,7 @@ static void dfps_copy (PixmapPtr dest,
>      RegionAppend(&info->updated_region, region);
>      RegionValidate(&info->updated_region, &throwaway_bool);
>      RegionUninit(region);
> +    RegionDestroy(region);
>  }
>  
>  static void dfps_done_copy (PixmapPtr dest)
> @@ -258,6 +260,7 @@ static Bool dfps_put_image (PixmapPtr dest, int x, int y,
> int w, int h,
>      RegionAppend(&info->updated_region, region);
>      RegionValidate(&info->updated_region, &throwaway_bool);
>      RegionUninit(region);
> +    RegionDestroy(region);
>  
>      /* We can avoid doing the put image ourselves, as the uxa driver
>         will fall back and do it for us if we return false */
> --
> 1.7.10.4
> 
> _______________________________________________
> Spice-devel mailing list
> Spice-devel at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/spice-devel
>