On Tue, Jan 10, 2017 at 07:44:32PM +0800, Steven Liu wrote:
> CID: 1396852
> check the devices_list alloc status,
> and release the devices_list when alloc devices error
> 
> Signed-off-by: Steven Liu <l...@chinaffmpeg.org>
> ---
>  cmdutils_opencl.c | 7 ++++++-
>  1 file changed, 6 insertions(+), 1 deletion(-)
> 
> diff --git a/cmdutils_opencl.c b/cmdutils_opencl.c
> index dd21344..5bbc8dc 100644
> --- a/cmdutils_opencl.c
> +++ b/cmdutils_opencl.c
> @@ -220,15 +220,20 @@ int opt_opencl_bench(void *optctx, const char *opt, 
> const char *arg)
>      OpenCLDeviceBenchmark *devices = NULL;
>      cl_platform_id platform;
>  
> -    av_opencl_get_device_list(&device_list);
> +    if (av_opencl_get_device_list(&device_list) < 0) {
> +        return AVERROR(ENOMEM);
> +    }

The error code from av_opencl_get_device_list() should be forwarded

thx

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

The educated differ from the uneducated as much as the living from the
dead. -- Aristotle 

Attachment: signature.asc
Description: Digital signature

_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Reply via email to