[Piglit] [PATCH] ext_image_dma_buf: Fix memory leak.

Tapani Pälli tapani.palli at intel.com
Tue Aug 18 23:05:47 PDT 2015


On 08/18/2015 07:46 AM, Vinson Lee wrote:
> Fix resource leak defect reported by Coverity.
>
> Signed-off-by: Vinson Lee <vlee at freedesktop.org>
> ---
>   tests/spec/ext_image_dma_buf_import/transcode-nv12-as-r8-gr88.c | 3 +++
>   1 file changed, 3 insertions(+)
>
> diff --git a/tests/spec/ext_image_dma_buf_import/transcode-nv12-as-r8-gr88.c b/tests/spec/ext_image_dma_buf_import/transcode-nv12-as-r8-gr88.c
> index 1e54855..e23e144 100644
> --- a/tests/spec/ext_image_dma_buf_import/transcode-nv12-as-r8-gr88.c
> +++ b/tests/spec/ext_image_dma_buf_import/transcode-nv12-as-r8-gr88.c
> @@ -294,8 +294,11 @@ piglit_display(void)
>   	piglit_tolerance[2] = 0.05;
>   	if (!piglit_probe_image_rgba(0, 0, piglit_width, piglit_height,
>   				     ref_rgba_image)) {
> +		free(ref_rgba_image);
>   		return PIGLIT_FAIL;
>   	}
>   
> +	free(ref_rgba_image);
> +
>   	return PIGLIT_PASS;
>   }

There are 2 more exit paths before these with piglit_report_result, they 
are leaking too.

// Tapani



More information about the Piglit mailing list