Re: [PATCH libdrm] xf86drm: Fix possible memory leak with drmModeGetPropertyPtr()

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

 



On Monday, 2019-04-29 18:10:52 +0900, Seung-Woo Kim wrote:
> In drmModeGetPropertyPtr(), from upper error path, it calls free
> but with just next error path, it does not call. Fix the possible
> memory leak.
> 
> Signed-off-by: Seung-Woo Kim <sw0312.kim@xxxxxxxxxxx>

Reviewed-by: Eric Engestrom <eric.engestrom@xxxxxxxxx>
and pushed, thanks!

> ---
>  xf86drmMode.c |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
> 
> diff --git a/xf86drmMode.c b/xf86drmMode.c
> index c878d9e..207d7be 100644
> --- a/xf86drmMode.c
> +++ b/xf86drmMode.c
> @@ -650,7 +650,7 @@ drm_public drmModePropertyPtr drmModeGetProperty(int fd, uint32_t property_id)
>  	}
>  
>  	if (!(r = drmMalloc(sizeof(*r))))
> -		return NULL;
> +		goto err_allocs;
>  
>  	r->prop_id = prop.prop_id;
>  	r->count_values = prop.count_values;
> -- 
> 1.7.4.1
> 
> _______________________________________________
> dri-devel mailing list
> dri-devel@xxxxxxxxxxxxxxxxxxxxx
> https://lists.freedesktop.org/mailman/listinfo/dri-devel
_______________________________________________
dri-devel mailing list
dri-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/dri-devel




[Index of Archives]     [Linux DRI Users]     [Linux Intel Graphics]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux