[Piglit] [PATCH] cl-api-create-sampler: Fix memory leak.

Jan Vesely jan.vesely at rutgers.edu
Sat Feb 7 12:10:13 PST 2015


On Fri, 2015-02-06 at 18:55 -0800, Vinson Lee wrote:
> Fix resource leak defect reported by Coverity.
> 
> Signed-off-by: Vinson Lee <vlee at freedesktop.org>
the function could be cleaned up a bit by using temp variable, but that
can be done later.

Reviewed-by: Jan Vesely <jan.vesely at rutgers.edu>

> ---
>  tests/cl/api/create-sampler.c | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/tests/cl/api/create-sampler.c b/tests/cl/api/create-sampler.c
> index 99423a3..dcdef05 100644
> --- a/tests/cl/api/create-sampler.c
> +++ b/tests/cl/api/create-sampler.c
> @@ -47,6 +47,7 @@ static bool context_has_image_support(const piglit_cl_context ctx)
>  			free(image_support);
>  			return true;
>  		}
> +		free(image_support);
>  	}
>  	return false;
>  }


-- 
Jan Vesely <jan.vesely at rutgers.edu>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: This is a digitally signed message part
URL: <http://lists.freedesktop.org/archives/piglit/attachments/20150207/6bc25908/attachment.sig>


More information about the Piglit mailing list