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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



ack, thanks

----- Original Message -----
> 
> Signed-off-by: Jeremy White <jwhite@xxxxxxxxxxxxxxx>
> ---
>  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@xxxxxxxxxxxxxxxxxxxxx
> http://lists.freedesktop.org/mailman/listinfo/spice-devel
> 
_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/spice-devel




[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]     [Monitors]