[FFmpeg-devel] Patch to libavc/opus to create extradata if missing

2020-12-27 Thread Jonathan Baudanza
This patch to libavcodec/opus.c will create the opus extradata if it is missing. This is required when muxing opus data from an RTP demuxer. Without this patch, the opux muxer will fail with a "No extradata present" error. This issue was first reported by Juan Navarro here: http://www.ffmpeg-a

Re: [FFmpeg-devel] [PATCH V2] libavcodec/qsvenc: add low latency P-pyramid support for qsv

2020-12-27 Thread Chen, Wenbin
> -Original Message- > From: Xiang, Haihao > Sent: Monday, December 28, 2020 10:40 AM > To: ffmpeg-devel@ffmpeg.org > Cc: Chen, Wenbin > Subject: Re: [FFmpeg-devel] [PATCH V2] libavcodec/qsvenc: add low latency > P-pyramid support for qsv > > > > From: Wenbinc-Bin > > > > Add low lat

Re: [FFmpeg-devel] [PATCH] libavcodec/qsvenc: add low latency P-pyramid support for qsv

2020-12-27 Thread Chen, Wenbin
> -Original Message- > From: Linjie Fu > Sent: Thursday, December 10, 2020 3:49 PM > To: FFmpeg development discussions and patches de...@ffmpeg.org> > Cc: Chen, Wenbin > Subject: Re: [FFmpeg-devel] [PATCH] libavcodec/qsvenc: add low latency P- > pyramid support for qsv > > On Thu, Dec

[FFmpeg-devel] [PATCH] avformat/mux: rewrite guessing the packet duration

2020-12-27 Thread smarsufan
From: smarsufan --- libavformat/mux.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavformat/mux.c b/libavformat/mux.c index 84c56ac6ba..3356ba6bf5 100644 --- a/libavformat/mux.c +++ b/libavformat/mux.c @@ -620,7 +620,7 @@ static int compute_muxer_pkt_fields(AVFormatCont

Re: [FFmpeg-devel] [PATCH V2] libavcodec/qsvenc: add low latency P-pyramid support for qsv

2020-12-27 Thread Xiang, Haihao
> From: Wenbinc-Bin > > Add low latency P-pyramid support for qsv, and it relates to a new > command line parameter "-p_strategy". To enable this flag, user also > need to set "-bf" to -1 or 0. What's the difference between v1 and v2? It would be better to add some comment about the change by u

Re: [FFmpeg-devel] [PATCH 8/8] vf_dnn_processing.c: add async support

2020-12-27 Thread Guo, Yejun
> -Original Message- > From: Guo, Yejun > Sent: 2020年12月22日 15:48 > To: ffmpeg-devel@ffmpeg.org > Cc: Guo, Yejun > Subject: [PATCH 8/8] vf_dnn_processing.c: add async support > > Signed-off-by: Xie, Lin > Signed-off-by: Wu Zhiwen > Signed-off-by: Guo, Yejun > --- > doc/filters.texi

Re: [FFmpeg-devel] [PATCH v2 3/3] avcodec/codec_desc: describe AV_CODEC_ID_MPEG2TS

2020-12-27 Thread Aman Karmani
Going to merge this patchset in a few days if no one objects. On Tue, Dec 22, 2020 at 2:47 PM Aman Karmani wrote: > From: Aman Karmani > > Signed-off-by: Aman Karmani > --- > libavcodec/codec_desc.c | 7 +++ > 1 file changed, 7 insertions(+) > > diff --git a/libavcodec/codec_desc.c b/liba

Re: [FFmpeg-devel] [PATCH 1/6] avformat/mxfdec: Do not clear arrays in mxf_read_index_entry_array()

2020-12-27 Thread Andreas Rheinhardt
Michael Niedermayer: > They are written too immediately, so it should not be needed. > > Signed-off-by: Michael Niedermayer > --- > libavformat/mxfdec.c | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) > > diff --git a/libavformat/mxfdec.c b/libavformat/mxfdec.c > index 1f79f3d3cd.

Re: [FFmpeg-devel] [PATCH 1/6] avformat/mxfdec: Do not clear arrays in mxf_read_index_entry_array()

2020-12-27 Thread James Almer
On 12/27/2020 2:49 PM, Michael Niedermayer wrote: They are written too immediately, so it should not be needed. Signed-off-by: Michael Niedermayer --- libavformat/mxfdec.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/libavformat/mxfdec.c b/libavformat/mxfdec.c in

Re: [FFmpeg-devel] [PATCH 6/6] avformat/mpegts: Fix argument type for av_log

2020-12-27 Thread Michael Niedermayer
On Sun, Dec 27, 2020 at 06:58:00PM +0100, Marton Balint wrote: > > > On Sun, 27 Dec 2020, Michael Niedermayer wrote: > > > Signed-off-by: Michael Niedermayer > > --- > > libavformat/mpegts.c | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/libavformat/mpegts.c b/li

Re: [FFmpeg-devel] [PATCH 7/7] avcodec/tiff: Disallow striped and tiled tiffs except for DNG

2020-12-27 Thread Michael Niedermayer
On Fri, Nov 06, 2020 at 12:11:10AM +0100, Michael Niedermayer wrote: > strips + tiles is not allowed in TIFF > DNG uses a separate codepath > > Regression since da5b3d002862d1e105002a6dc1567e6551860896. > > Fixes: NULL pointer dereference > Fixes: poc1 > > Found-by: 1vanChen of NSFOCUS Security

Re: [FFmpeg-devel] [PATCH 6/6] avformat/mpegts: Fix argument type for av_log

2020-12-27 Thread Marton Balint
On Sun, 27 Dec 2020, Michael Niedermayer wrote: Signed-off-by: Michael Niedermayer --- libavformat/mpegts.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavformat/mpegts.c b/libavformat/mpegts.c index cc292ab929..efff5130de 100644 --- a/libavformat/mpegts.c +++ b/libav

[FFmpeg-devel] [PATCH 4/6] tools/target_dem_fuzzer.c: Decrease maxblocks

2020-12-27 Thread Michael Niedermayer
Fixes: Timeout Fixes: 28606/clusterfuzz-testcase-minimized-ffmpeg_dem_FRM_fuzzer-5123311424110592 Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg Signed-off-by: Michael Niedermayer --- tools/target_dem_fuzzer.c | 2 +- 1 file changed, 1 inser

[FFmpeg-devel] [PATCH 2/6] avformat/vividas: Check number of audio channels

2020-12-27 Thread Michael Niedermayer
Fixes: division by 0 Fixes: 28597/clusterfuzz-testcase-minimized-ffmpeg_dem_VIVIDAS_fuzzer-5752201490333696 Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg Signed-off-by: Michael Niedermayer --- libavformat/vividas.c | 2 +- 1 file changed, 1

[FFmpeg-devel] [PATCH 1/6] avformat/mxfdec: Do not clear arrays in mxf_read_index_entry_array()

2020-12-27 Thread Michael Niedermayer
They are written too immediately, so it should not be needed. Signed-off-by: Michael Niedermayer --- libavformat/mxfdec.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/libavformat/mxfdec.c b/libavformat/mxfdec.c index 1f79f3d3cd..f5472525ff 100644 --- a/libavformat/mx

[FFmpeg-devel] [PATCH 6/6] avformat/mpegts: Fix argument type for av_log

2020-12-27 Thread Michael Niedermayer
Signed-off-by: Michael Niedermayer --- libavformat/mpegts.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavformat/mpegts.c b/libavformat/mpegts.c index cc292ab929..efff5130de 100644 --- a/libavformat/mpegts.c +++ b/libavformat/mpegts.c @@ -3140,7 +3140,7 @@ static int mp

[FFmpeg-devel] [PATCH 5/6] avcodec/alsdec: Fix integer overflow with quant_cof

2020-12-27 Thread Michael Niedermayer
Fixes: signed integer overflow: -210824 * 16384 cannot be represented in type 'int' Fixes: 28670/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_ALS_fuzzer-5682310846480384 Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg Signed-off-by: Micha

[FFmpeg-devel] [PATCH 3/6] avcodec/tiff: Check strip values when they will be used

2020-12-27 Thread Michael Niedermayer
Regression since: da5b3d002862d1e105002a6dc1567e6551860896 Fixes: Assertion buf_size >= 0 failed at libavcodec/bytestream.h:151 Fixes: 28599/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_TIFF_fuzzer-5362335027953664 Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tre

Re: [FFmpeg-devel] [PATCH] libswscale/swscale.c: Clarify what exactly 'data is not aligned' to

2020-12-27 Thread Carl Eugen Hoyos
> Am 27.12.2020 um 09:36 schrieb Franziska Thul : > > If "bytes" or "pixels" doesn't matter from the user's perspective, I'd stick > with the latter, even if it's not exactly right regarding the internal > workings of ffmpeg. It does matter because while data has to be 16-bytes aligned, thi

Re: [FFmpeg-devel] [PATCH] libswscale/swscale.c: Clarify what exactly 'data is not aligned' to

2020-12-27 Thread Franziska Thul
Michael Niedermayer wrote: > > libswscale/swscale.c emits a Warning that 'data is not aligned', but > > doesn't explain > > why, leaving users without any clue on how to address this issue. > > This patch simply adds that data is not aligned 'to 16 pixel boundaries'. > all the alignment is by by