[FFmpeg-devel] [PATCH 6/8] avformat/rmdec: Fix codecdata_length overflow check

2020-12-18 Thread Michael Niedermayer
Fixes: signed integer overflow: 2147483647 + 64 cannot be represented in type 'int' Fixes: 28509/clusterfuzz-testcase-minimized-ffmpeg_dem_IVR_fuzzer-6310969680723968 Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg Signed-off-by: Michael Niede

[FFmpeg-devel] [PATCH 4/8] avcodec/ffv1dec: Fix off by 1 error with quant tables

2020-12-18 Thread Michael Niedermayer
Fixes: assertion failure Fixes: 28447/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_FFV1_fuzzer-5369575948550144 Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg Signed-off-by: Michael Niedermayer --- libavcodec/ffv1dec.c | 2 +- 1 file ch

[FFmpeg-devel] [PATCH 2/8] avformat/mpegts: simplify nb_packets code

2020-12-18 Thread Michael Niedermayer
Signed-off-by: Michael Niedermayer --- libavformat/mpegts.c | 7 ++- 1 file changed, 2 insertions(+), 5 deletions(-) diff --git a/libavformat/mpegts.c b/libavformat/mpegts.c index 1122455f66..cc292ab929 100644 --- a/libavformat/mpegts.c +++ b/libavformat/mpegts.c @@ -3090,7 +3090,6 @@ static

[FFmpeg-devel] [PATCH 1/8] avformat/mpegts: Increase pcr_incr width to 64bit

2020-12-18 Thread Michael Niedermayer
Fixes: division by zero Fixes: 26459/clusterfuzz-testcase-minimized-ffmpeg_dem_MPEGTSRAW_fuzzer-5666350112178176 Fixes: 28154/clusterfuzz-testcase-minimized-ffmpeg_dem_MPEGTSRAW_fuzzer-5195728439476224 Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/projects/

[FFmpeg-devel] [PATCH 3/8] avcodec/simple_idct: Fix undefined integer overflow in idct4row()

2020-12-18 Thread Michael Niedermayer
Fixes: signed integer overflow: -1498310196 - 902891776 cannot be represented in type 'int' Fixes: 28445/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_VC1IMAGE_fuzzer-5075163389493248 Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg Signed-

Re: [FFmpeg-devel] [PATCH 5/8] avcodec/hevc_ps: check scaling_list_dc_coef

2020-12-18 Thread James Almer
On 12/18/2020 8:22 PM, Michael Niedermayer wrote: Fixes: signed integer overflow: 2147483640 + 8 cannot be represented in type 'int' Fixes: 28449/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_HEVC_fuzzer-5686013259284480 Signed-off-by: Michael Niedermayer --- libavcodec/hevc_ps.c | 6 ++

[FFmpeg-devel] [PATCH 5/8] avcodec/hevc_ps: check scaling_list_dc_coef

2020-12-18 Thread Michael Niedermayer
Fixes: signed integer overflow: 2147483640 + 8 cannot be represented in type 'int' Fixes: 28449/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_HEVC_fuzzer-5686013259284480 Signed-off-by: Michael Niedermayer --- libavcodec/hevc_ps.c | 6 +- 1 file changed, 5 insertions(+), 1 deletion(-)

[FFmpeg-devel] [PATCH 8/8] avformat/nutdec: Check timebase count against main header length

2020-12-18 Thread Michael Niedermayer
Fixes: Timeout (long -> 3ms) Fixes: 28514/clusterfuzz-testcase-minimized-ffmpeg_dem_NUT_fuzzer-6078669009321984 Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg Signed-off-by: Michael Niedermayer --- libavformat/nutdec.c | 8 1 file c

[FFmpeg-devel] [PATCH 7/8] avformat/aviobuf: Check for overflow in ffio_read_varlen()

2020-12-18 Thread Michael Niedermayer
No testcase Signed-off-by: Michael Niedermayer --- libavformat/aviobuf.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/libavformat/aviobuf.c b/libavformat/aviobuf.c index 78cc60b2ae..7730547106 100644 --- a/libavformat/aviobuf.c +++ b/libavformat/aviobuf.c @@ -917,6 +917,8 @@ uint64_t ff

Re: [FFmpeg-devel] [FFmpeg-cvslog] avcodec/dpx: fix use of uninitialised value

2020-12-18 Thread James Almer
On 12/18/2020 6:22 PM, Andriy Gelman wrote: On Fri, 18. Dec 18:22, James Almer wrote: ffmpeg | branch: master | James Almer | Fri Dec 18 14:31:47 2020 -0300| [20b09b20a942d4aad38f9fa1324b713168d3db9a] | committer: James Almer avcodec/dpx: fix use of uninitialised value Found by Valgrind. Re

Re: [FFmpeg-devel] [PATCH] avformat/libamqp: add option vhost

2020-12-18 Thread Andriy Gelman
On Thu, 17. Dec 21:34, Florian Levis wrote: > From: Florian Levis > > Add option vhost to allow publishing on other > vhost than default '/' > > Signed-off-by: Florian Levis > --- > doc/protocols.texi| 3 +++ > libavformat/libamqp.c | 4 +++- > 2 files changed, 6 insertions(+), 1 deletion(

Re: [FFmpeg-devel] [FFmpeg-cvslog] avcodec/dpx: fix use of uninitialised value

2020-12-18 Thread Andriy Gelman
On Fri, 18. Dec 18:22, James Almer wrote: > ffmpeg | branch: master | James Almer | Fri Dec 18 > 14:31:47 2020 -0300| [20b09b20a942d4aad38f9fa1324b713168d3db9a] | committer: > James Almer > > avcodec/dpx: fix use of uninitialised value > > Found by Valgrind. > > Reviewed-by: Paul B Mahol > S

Re: [FFmpeg-devel] [PATCH] avcodec/dpx: fix use of uninitialised value

2020-12-18 Thread James Almer
On 12/18/2020 3:03 PM, Paul B Mahol wrote: LGTM Applied. On Fri, Dec 18, 2020 at 6:41 PM James Almer wrote: Found by Valgrind. Signed-off-by: James Almer --- libavcodec/dpx.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/libavcodec/dpx.c b/libavcodec/dpx.c index bd431ccbcb..1

Re: [FFmpeg-devel] [PATCH] avcodec/dpx: fix use of uninitialised value

2020-12-18 Thread Paul B Mahol
LGTM On Fri, Dec 18, 2020 at 6:41 PM James Almer wrote: > Found by Valgrind. > > Signed-off-by: James Almer > --- > libavcodec/dpx.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/libavcodec/dpx.c b/libavcodec/dpx.c > index bd431ccbcb..12bc165415 100644 > --- a/libavcodec/dpx.c > +

[FFmpeg-devel] [PATCH] avcodec/dpx: fix use of uninitialised value

2020-12-18 Thread James Almer
Found by Valgrind. Signed-off-by: James Almer --- libavcodec/dpx.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/libavcodec/dpx.c b/libavcodec/dpx.c index bd431ccbcb..12bc165415 100644 --- a/libavcodec/dpx.c +++ b/libavcodec/dpx.c @@ -333,10 +333,12 @@ static int decode_frame(AVCodecCont

Re: [FFmpeg-devel] [PATCH 1/4] avcodec/h264_sei: refactor parsing User Data Registered ITU-T T35 SEI messages

2020-12-18 Thread James Almer
On 12/18/2020 12:51 PM, Jan Ekström wrote: On Tue, Dec 8, 2020 at 9:44 PM James Almer wrote: Signed-off-by: James Almer --- libavcodec/h264_sei.c | 31 ++- 1 file changed, 22 insertions(+), 9 deletions(-) Set generally OK, just some minor things: - probably AV

Re: [FFmpeg-devel] [PATCH 1/4] avcodec/h264_sei: refactor parsing User Data Registered ITU-T T35 SEI messages

2020-12-18 Thread Jan Ekström
On Tue, Dec 8, 2020 at 9:44 PM James Almer wrote: > > Signed-off-by: James Almer > --- > libavcodec/h264_sei.c | 31 ++- > 1 file changed, 22 insertions(+), 9 deletions(-) Set generally OK, just some minor things: - probably AVERROR(EINVAL) usage should be AVERROR_IN

[FFmpeg-devel] [PATCH] dnn: add NV12 pixel format support

2020-12-18 Thread Ting Fu
Signed-off-by: Ting Fu --- libavfilter/dnn/dnn_io_proc.c | 2 ++ libavfilter/vf_dnn_processing.c | 30 +- 2 files changed, 23 insertions(+), 9 deletions(-) diff --git a/libavfilter/dnn/dnn_io_proc.c b/libavfilter/dnn/dnn_io_proc.c index c9b49be3bd..2744cb6502 10064