Re: [FFmpeg-devel] [PATCH] avfilter/vf_avgblur_opencl: fix error when clSetKernelArg fails

2018-03-26 Thread Mark Thompson
On 26/03/18 01:08, dylanf...@gmail.com wrote: > From: drfer3 > > Fixes Coverity CID 1430382 > --- > libavfilter/vf_avgblur_opencl.c | 6 ++ > 1 file changed, 6 insertions(+) LGTM, applied this along with the image size one. Thanks, - Mark

[FFmpeg-devel] [PATCH] avfilter/vf_avgblur_opencl: fix error when clSetKernelArg fails

2018-03-25 Thread dylanf123
From: drfer3 Fixes Coverity CID 1430382 --- libavfilter/vf_avgblur_opencl.c | 6 ++ 1 file changed, 6 insertions(+) diff --git a/libavfilter/vf_avgblur_opencl.c b/libavfilter/vf_avgblur_opencl.c index 5ee66c0ba2..53db83c21f 100644 ---

Re: [FFmpeg-devel] [PATCH] avfilter/vf_avgblur_opencl: fix error when clSetKernelArg fails

2018-03-25 Thread Mark Thompson
On 25/03/18 13:42, dylanf...@gmail.com wrote: > From: drfer3 > > Fixes Coverity CID 1430382 > ---> Following is a patch attempting to fix the err issue. It returns -1 if > any > clSetKernelArg() fails. Is this good, or should I be using a different > return value for

[FFmpeg-devel] [PATCH] avfilter/vf_avgblur_opencl: fix error when clSetKernelArg fails

2018-03-25 Thread dylanf123
From: drfer3 Fixes Coverity CID 1430382 --- libavfilter/vf_avgblur_opencl.c | 6 ++ 1 file changed, 6 insertions(+) diff --git a/libavfilter/vf_avgblur_opencl.c b/libavfilter/vf_avgblur_opencl.c index 5ee66c0ba2..09caa1fd4f 100644 ---