Re: [FFmpeg-devel] [PATCH 1/3] avcodec/magicyuv: Check bits left in flags&1 branch

2018-06-24 Thread Paul B Mahol
On 6/24/18, Michael Niedermayer wrote: > Fixes: Timeout > Fixes: > 8690/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_MAGICYUV_fuzzer-6542020913922048 > > Found-by: continuous fuzzing process > https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg > Signed-off-by: Michael Niedermayer

[FFmpeg-devel] [PATCH 4/5] lavf/mpjpegdec: Mark local variable static

2018-06-24 Thread Mark Thompson
--- libavformat/mpjpegdec.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavformat/mpjpegdec.c b/libavformat/mpjpegdec.c index 83aa70d0c9..64d880a981 100644 --- a/libavformat/mpjpegdec.c +++ b/libavformat/mpjpegdec.c @@ -375,7 +375,7 @@ static int mpjpeg_read_packet(AVForm

[FFmpeg-devel] [PATCH 3/5] lavfi/framesync: Add namespace prefix to framesync_get_class

2018-06-24 Thread Mark Thompson
--- libavfilter/framesync.c | 2 +- libavfilter/framesync.h | 6 +++--- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/libavfilter/framesync.c b/libavfilter/framesync.c index da12c58a61..22d3f091a3 100644 --- a/libavfilter/framesync.c +++ b/libavfilter/framesync.c @@ -61,7 +61,7 @@

[FFmpeg-devel] [PATCH 2/5] lavfi/vf_tonemap_opencl: Mark local tables static

2018-06-24 Thread Mark Thompson
--- libavfilter/vf_tonemap_opencl.c | 14 +++--- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git a/libavfilter/vf_tonemap_opencl.c b/libavfilter/vf_tonemap_opencl.c index b4ef7f269b..11978e64a6 100644 --- a/libavfilter/vf_tonemap_opencl.c +++ b/libavfilter/vf_tonemap_opencl.c @

Re: [FFmpeg-devel] [PATCH 3/5] lavfi/framesync: Add namespace prefix to framesync_get_class

2018-06-24 Thread Nicolas George
Mark Thompson (2018-06-24): > --- > libavfilter/framesync.c | 2 +- > libavfilter/framesync.h | 6 +++--- > 2 files changed, 4 insertions(+), 4 deletions(-) LGTM, thanks. Regards, -- Nicolas George signature.asc Description: Digital signature ___

[FFmpeg-devel] [PATCH 1/5] lavfi/colorspace: Add namespace prefix to global functions

2018-06-24 Thread Mark Thompson
--- libavfilter/colorspace.c| 13 +++-- libavfilter/colorspace.h| 10 ++ libavfilter/vf_colorspace.c | 22 +++--- libavfilter/vf_tonemap_opencl.c | 8 4 files changed, 28 insertions(+), 25 deletions(-) diff --git a/libavfilter/colorspa

[FFmpeg-devel] [PATCH 5/5] fate: Add test for namespace prefixes in libraries

2018-06-24 Thread Mark Thompson
Ensures that there are no external symbols in the built static libraries which do not have suitable namespace prefixes. --- Not sure whether this will actually work on all platforms? (Only tested with Linux/binutils.) tests/Makefile | 1 + tests/fate/namespace.mak | 20 ++

Re: [FFmpeg-devel] [PATCH 3/5] lavfi/framesync: Add namespace prefix to framesync_get_class

2018-06-24 Thread Mark Thompson
On 24/06/18 19:29, Nicolas George wrote: > Mark Thompson (2018-06-24): >> --- >> libavfilter/framesync.c | 2 +- >> libavfilter/framesync.h | 6 +++--- >> 2 files changed, 4 insertions(+), 4 deletions(-) > > LGTM, thanks. Applied. Thanks, - Mark ___

Re: [FFmpeg-devel] [PATCH] fftools/ffmpeg: check sseof value and clash with ss

2018-06-24 Thread Gyan Doshi
Will push in a day if no comments. (fixed the missing loglevel locally) Gyan ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH 09/12] avformat/mxfdec: add support for clip wrapped essences

2018-06-24 Thread Marton Balint
On Thu, 14 Jun 2018, Marton Balint wrote: On Thu, 14 Jun 2018, Tomas Härdin wrote: ons 2018-06-13 klockan 22:11 +0200 skrev Marton Balint: On Wed, 13 Jun 2018, Tomas Härdin wrote: > sön 2018-06-10 klockan 12:36 +0200 skrev Marton Balint: > > Also use common code with opAtom. > > > > Fi

Re: [FFmpeg-devel] [PATCH 2/2] lavf/vf_scale_amf: AMF scaler/colorspace converter filter implementation

2018-06-24 Thread Mark Thompson
On 19/06/18 15:34, Alexander Kravchenko wrote: > Hi Mark. > Thanks for your review and comments. > See my comments and question bellow > >> -Original Message- >> From: ffmpeg-devel [mailto:ffmpeg-devel-boun...@ffmpeg.org] On Behalf Of >> Mark Thompson >> Sent: Tuesday, June 19, 2018 2:20

[FFmpeg-devel] [PATCH] avcodec/escape124: Check buf_size against num_superblocks

2018-06-24 Thread Michael Niedermayer
Fixes: Timeout Fixes: 8722/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_ESCAPE124_fuzzer-4843268402577408 Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg Signed-off-by: Michael Niedermayer --- libavcodec/escape124.c | 2 +- 1 file change

Re: [FFmpeg-devel] [PATCH] avcodec/escape124: Check buf_size against num_superblocks

2018-06-24 Thread James Almer
On 6/24/2018 6:18 PM, Michael Niedermayer wrote: > Fixes: Timeout > Fixes: > 8722/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_ESCAPE124_fuzzer-4843268402577408 > > Found-by: continuous fuzzing process > https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg > Signed-off-by: Michael

[FFmpeg-devel] [PATCH 1/5] avformat/mxfdec: add support for clip wrapped essences

2018-06-24 Thread Marton Balint
Also use common code with opAtom. Fixes ticket #2776. Partially fixes ticket #5671. Fixes ticket #5866. Signed-off-by: Marton Balint --- libavformat/mxfdec.c | 282 --- 1 file changed, 131 insertions(+), 151 deletions(-) diff --git a/libavformat/

[FFmpeg-devel] [PATCH 3/5] avformat/mxfdec: take into account index_edit_rate

2018-06-24 Thread Marton Balint
Fixes samples/ffmpeg-bugs/roundup/issue591/02785736.mxf Fixes samples/ffmpeg-bugs/trac/ticket1916/pcm_s24le_to_pcm_s16le.mxf Signed-off-by: Marton Balint --- libavformat/mxfdec.c | 19 ++- 1 file changed, 14 insertions(+), 5 deletions(-) diff --git a/libavformat/mxfdec.c b/libav

[FFmpeg-devel] [PATCH 4/5] avformat/mxfdec: guess constant byte count indexes based on track duration

2018-06-24 Thread Marton Balint
For clip wrapped essences this should work. Also, since index_edit_rate can now be different from track edit rate, remove overriding track edit rate. Signed-off-by: Marton Balint --- libavformat/mxfdec.c | 20 +++- 1 file changed, 15 insertions(+), 5 deletions(-) diff --git a/li

[FFmpeg-devel] [PATCH 2/5] avformat/mxfdec: fix indentation and rename mxf_read_packet_old

2018-06-24 Thread Marton Balint
Signed-off-by: Marton Balint --- libavformat/mxfdec.c | 34 +- 1 file changed, 17 insertions(+), 17 deletions(-) diff --git a/libavformat/mxfdec.c b/libavformat/mxfdec.c index c25595a91f..fb03cb97f1 100644 --- a/libavformat/mxfdec.c +++ b/libavformat/mxfdec.c @@ -

[FFmpeg-devel] [PATCH 5/5] avformat/mxfdec: add support for opAtom without index

2018-06-24 Thread Marton Balint
Clip wrapped code is capable of doing some magic for such files. Signed-off-by: Marton Balint --- libavformat/mxfdec.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavformat/mxfdec.c b/libavformat/mxfdec.c index 309419bbc2..b76beb962f 100644 --- a/libavformat/mxfdec.c ++

Re: [FFmpeg-devel] [PATCH 1/3] avcodec/magicyuv: Check bits left in flags&1 branch

2018-06-24 Thread Michael Niedermayer
On Sun, Jun 24, 2018 at 10:20:23AM +0200, Paul B Mahol wrote: > On 6/24/18, Michael Niedermayer wrote: > > Fixes: Timeout > > Fixes: > > 8690/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_MAGICYUV_fuzzer-6542020913922048 > > > > Found-by: continuous fuzzing process > > https://github.com/googl

[FFmpeg-devel] [PATCH] lavfi: add sobel, prewitt, roberts filters

2018-06-24 Thread Danil Iashchenko
Add opencl version of sobel, prewitt, roberts filters. --- configure | 3 + libavfilter/Makefile| 8 +- libavfilter/allfilters.c| 3 + libavfilter/opencl/convolution.cl | 82 ++ libavfilter/vf_convolution_opencl.c | 306 +++

[FFmpeg-devel] [PATCH] avcodec/escape124: Check buf_size against num_superblocks

2018-06-24 Thread Michael Niedermayer
Fixes: Timeout Fixes: 8722/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_ESCAPE124_fuzzer-4843268402577408 Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg Signed-off-by: Michael Niedermayer --- libavcodec/escape124.c | 6 +- 1 file ch

Re: [FFmpeg-devel] [PATCH] avcodec/escape124: Check buf_size against num_superblocks

2018-06-24 Thread Michael Niedermayer
On Sun, Jun 24, 2018 at 07:00:25PM -0300, James Almer wrote: > On 6/24/2018 6:18 PM, Michael Niedermayer wrote: > > Fixes: Timeout > > Fixes: > > 8722/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_ESCAPE124_fuzzer-4843268402577408 > > > > Found-by: continuous fuzzing process > > https://gith

Re: [FFmpeg-devel] [PATCH] avcodec/h264_parser: Reduce needed history for parsing mb index

2018-06-24 Thread Michael Niedermayer
On Fri, Jun 22, 2018 at 09:45:59PM +0200, Michael Niedermayer wrote: > This fixes a bug/regression with very small packets > Fixes: output_file > > Reported-by: Thierry Foucu > Signed-off-by: Michael Niedermayer > --- > libavcodec/h264_parser.c | 19 +++ > 1 file changed, 11 ins

Re: [FFmpeg-devel] [PATCH] fftools/ffmpeg: check sseof value and clash with ss

2018-06-24 Thread Michael Niedermayer
On Sat, Jun 23, 2018 at 10:17:45AM +0530, Gyan Doshi wrote: > > Added another check and corrected an oversight in a log message. > ffmpeg_opt.c | 16 +++- > 1 file changed, 15 insertions(+), 1 deletion(-) > 663aeb89592e5538ac36e738815b105ab0708d6f > v2-0001-fftools-ffmpeg-check-s

Re: [FFmpeg-devel] [PATCH 2/5] lavfi/vf_tonemap_opencl: Mark local tables static

2018-06-24 Thread Song, Ruiling
> -Original Message- > From: ffmpeg-devel [mailto:ffmpeg-devel-boun...@ffmpeg.org] On Behalf Of > Mark Thompson > Sent: Monday, June 25, 2018 2:26 AM > To: ffmpeg-devel@ffmpeg.org > Subject: [FFmpeg-devel] [PATCH 2/5] lavfi/vf_tonemap_opencl: Mark local > tables static > > --- > libavfi

Re: [FFmpeg-devel] [PATCH 1/5] lavfi/colorspace: Add namespace prefix to global functions

2018-06-24 Thread Song, Ruiling
> -Original Message- > From: ffmpeg-devel [mailto:ffmpeg-devel-boun...@ffmpeg.org] On Behalf Of > Mark Thompson > Sent: Monday, June 25, 2018 2:26 AM > To: ffmpeg-devel@ffmpeg.org > Subject: [FFmpeg-devel] [PATCH 1/5] lavfi/colorspace: Add namespace prefix to > global functions > > --- >

Re: [FFmpeg-devel] [PATCH v2 3/4] hwcontext_opencl: Remove the opencl_device_init in opencl_device_derive

2018-06-24 Thread myp...@gmail.com
On Thu, Jun 21, 2018 at 7:50 PM Carl Eugen Hoyos wrote: > > 2018-06-21 13:45 GMT+02:00, Jun Zhao : > > In opencl device derived case, don't need to call opencl_device_init. > > > > Signed-off-by: Jun Zhao > > --- > > libavutil/hwcontext_opencl.c | 7 ++- > > 1 file changed, 2 insertions(+),

Re: [FFmpeg-devel] [PATCH] fftools/ffmpeg: check sseof value and clash with ss

2018-06-24 Thread Gyan Doshi
On 25-06-2018 06:39 AM, Michael Niedermayer wrote: arguments types are wrong } + timestamp = (o->start_time == AV_NOPTS_VALUE) ? 0 : o->start_time; unrelated, this should not be in this patch Revised. Will push soon. Thanks, Gyan From ca9d51bca4cb303897bf32ff3900fa36f3cf707

Re: [FFmpeg-devel] [PATCH 2/6] lavfi: add common Vulkan filtering code

2018-06-24 Thread Rostislav Pehlivanov
On 22 June 2018 at 15:43, James Almer wrote: > On 6/21/2018 1:55 PM, Rostislav Pehlivanov wrote: > > @@ -3462,12 +3465,12 @@ avcodec_select="null_bsf" > > avdevice_deps="avformat avcodec avutil" > > avdevice_suggest="libm" > > avfilter_deps="avutil" > > -avfilter_suggest="libm" > > +avfilter_s