Re: [FFmpeg-devel] [PATCH] ffprobe: add decode_error_flags

2021-05-27 Thread Tobias Rapp
On 25.05.2021 13:27, Nicolas George wrote: Michael Niedermayer (12021-05-25): Signed-off-by: Michael Niedermayer --- doc/ffprobe.xsd| 1 + fftools/ffprobe.c | 2 + tests/ref/fate/exif-image-embedded | 22 ++ tests/ref/fate/exif-image-jpg | 1

[FFmpeg-devel] [PATCH] avutil/mem: use GCC builtins to check for overflow in av_size_mult()

2021-05-27 Thread James Almer
Signed-off-by: James Almer --- libavutil/mem.h | 9 - 1 file changed, 8 insertions(+), 1 deletion(-) diff --git a/libavutil/mem.h b/libavutil/mem.h index e21a1feaae..c876111afb 100644 --- a/libavutil/mem.h +++ b/libavutil/mem.h @@ -674,11 +674,18 @@ void *av_dynarray2_add(void **tab_ptr,

[FFmpeg-devel] [PATCH] avformat/libsrt: log streamid in listener mode

2021-05-27 Thread Zhao Zhili
It's useful for test client which pass streamid to ffmpeg/ffplay. For example, use ffmpeg to test streamid support in VLC: ./ffmpeg -v info -re -i foo.mp4 -c copy -f mpegts -mode listener srt://127.0.0.1:9000 ./vlc srt://127.0.0.1:9000?streamid=foobar --- libavformat/libsrt.c | 6 ++ 1 file c

Re: [FFmpeg-devel] [PATCH 3/3] avformat/avio: Check av_opt_copy() for failure

2021-05-27 Thread Michael Niedermayer
On Thu, May 20, 2021 at 11:25:49PM +0200, Michael Niedermayer wrote: > Fixes: CID1477416 Unchecked return value > > Signed-off-by: Michael Niedermayer > --- > libavformat/avio.c | 7 +-- > 1 file changed, 5 insertions(+), 2 deletions(-) will apply [...] -- Michael GnuPG fingerprint: 9

Re: [FFmpeg-devel] [PATCH 2/3] Remove format (Alternative solution for CID1477423)

2021-05-27 Thread Michael Niedermayer
On Thu, May 20, 2021 at 11:25:48PM +0200, Michael Niedermayer wrote: > Signed-off-by: Michael Niedermayer > --- > libavformat/moflex.c | 3 --- > 1 file changed, 3 deletions(-) will combine with patch 1 and apply [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Re: [FFmpeg-devel] [PATCH 1/2] avcodec/error_resilience: cleanup mpeg2 handling

2021-05-27 Thread Michael Niedermayer
On Mon, May 24, 2021 at 10:43:54PM +0200, Michael Niedermayer wrote: > After this, the loop for the mpeg2 case is only executed when needed > > Signed-off-by: Michael Niedermayer > --- > libavcodec/error_resilience.c | 23 --- > 1 file changed, 12 insertions(+), 11 deletions(

Re: [FFmpeg-devel] [PATCH 2/2] avcodec/exr: x/ymax cannot be INT_MAX

2021-05-27 Thread Michael Niedermayer
On Wed, Apr 21, 2021 at 02:31:18PM +0200, Michael Niedermayer wrote: > The code uses x/ymax + 1 so the maximum is INT_MAX-1 > > Fixes: signed integer overflow: 2147483647 + 1 cannot be represented in type > 'int' > Fixes: > 33158/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_EXR_fuzzer-55454

Re: [FFmpeg-devel] [PATCH] avcodec/exr: More strictly check dc_count

2021-05-27 Thread Michael Niedermayer
On Tue, May 25, 2021 at 10:22:02PM +0200, Michael Niedermayer wrote: > Fixes: out of array access > Fixes: exr/deneme > > Found-by: Burak Çarıkçı > Signed-off-by: Michael Niedermayer > --- > libavcodec/exr.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) will apply [...] -- Mich

Re: [FFmpeg-devel] [PATCH] avfilter/yadif: Fix time base for large denominators

2021-05-27 Thread Michael Niedermayer
On Tue, May 25, 2021 at 11:57:08AM -0400, Tom Boshoven wrote: > This fixes an issue where the yadif filter could cause the timebase > denominator to overflow. > > Signed-off-by: Tom Boshoven > --- > libavfilter/vf_yadif.c | 3 +-- > 1 file changed, 1 insertion(+), 2 deletions(-) will apply th

Re: [FFmpeg-devel] [PATCH] avcodec: Pass the HDR10+ metadata to the packet side data in VP9 encoder

2021-05-27 Thread Mohammad Izadi
All done. On Wed, May 26, 2021 at 2:42 PM Lynne wrote: > May 26, 2021, 20:43 by izadi-at-google@ffmpeg.org: > > > HDR10+ metadata is stored in the bit stream for HEVC. The story is > different for VP9 and cannot store the metadata in the bit stream. HDR10+ > should be passed to packet side

[FFmpeg-devel] [PATCH] avcodec: Pass the HDR10+ metadata to the packet side data in VP9 encoder

2021-05-27 Thread Mohammad Izadi
HDR10+ metadata is stored in the bit stream for HEVC. The story is different for VP9 and cannot store the metadata in the bit stream. HDR10+ should be passed to packet side data an stored in the container (mkv) for VP9. This CL is taking HDR10+ from AVFrame side data in libvpxenc and is passing

Re: [FFmpeg-devel] [PATCH] avcodec: Pass the HDR10+ metadata to the packet side data in VP9 encoder

2021-05-27 Thread Mohammad Izadi
On Wed, May 26, 2021 at 9:17 PM James Zern wrote: > On Wed, May 26, 2021 at 6:35 PM Mohammad Izadi > wrote: > > [...] > > +static void add_hdr10_plus(AVFifoBuffer *fifo, struct FrameHDR10Plus > *data) > > +{ > > +av_fifo_grow(fifo, sizeof(FrameHDR10Plus)); > > This return should be checked.

Re: [FFmpeg-devel] [PATCH] ffprobe: add decode_error_flags

2021-05-27 Thread Marton Balint
On Thu, 27 May 2021, Tobias Rapp wrote: On 25.05.2021 13:27, Nicolas George wrote: Michael Niedermayer (12021-05-25): Signed-off-by: Michael Niedermayer --- doc/ffprobe.xsd| 1 + fftools/ffprobe.c | 2 + tests/ref/fate/exif-image-embedded | 22

[FFmpeg-devel] [PATCH] lavfi/dnn_backend_openvino.c: Correct Pointer Type while Freeing

2021-05-27 Thread Shubhanshu Saxena
This commit corrects the type of pointer of elements from the inference queue in ff_dnn_free_model_ov. Signed-off-by: Shubhanshu Saxena --- libavfilter/dnn/dnn_backend_openvino.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavfilter/dnn/dnn_backend_openvino.c b/libavfi

Re: [FFmpeg-devel] [PATCH] avisynth.c corrected interlace detection

2021-05-27 Thread Stephen Hutchinson
On 5/24/2021 11:03 AM, emcodem wrote: Sorry for the delay on this, should have corrected it much earlier. There was some confusion in the interlaced analysis. From 3rdparty decoders perspective, a clip can only be interlaced when it is NOT field_based. This is because in a field_based clip, the

Re: [FFmpeg-devel] [PATCH] lavfi/dnn_backend_openvino.c: Correct Pointer Type while Freeing

2021-05-27 Thread Guo, Yejun
> -Original Message- > From: ffmpeg-devel On Behalf Of > Shubhanshu Saxena > Sent: 2021年5月28日 2:06 > To: ffmpeg-devel@ffmpeg.org > Cc: Shubhanshu Saxena > Subject: [FFmpeg-devel] [PATCH] lavfi/dnn_backend_openvino.c: Correct > Pointer Type while Freeing > > This commit corrects the typ

[FFmpeg-devel] [PATCH 2/3] libavcodec/mips: Fix build errors reported by clang

2021-05-27 Thread Jin Bo
Clang is more strict on the type of asm operands, float or double type variable should use constraint 'f', integer variable should use constraint 'r'. Signed-off-by: Jin Bo --- libavcodec/mips/constants.c | 89 +++-- libavcodec/mips/constants.h | 88 +++-- libavcodec/

[FFmpeg-devel] [PATCH 3/3] libavcodec/mips: Fix fate errors reported by clang

2021-05-27 Thread Jin Bo
The data width of gsldrc1/gsldlc1 should be 8 bytes wide. Signed-off-by: Jin Bo --- libavcodec/mips/vp9_mc_mmi.c | 16 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/libavcodec/mips/vp9_mc_mmi.c b/libavcodec/mips/vp9_mc_mmi.c index fa65ff5..812f7a6 100644 --- a/li

Re: [FFmpeg-devel] [PATCH 00/27] Metadata handling in CBS

2021-05-27 Thread Kenny McClive
Hi, Are there any plans to get this (or even a similar patch) merged? I was excited to see this initially, but it has been six months now without a word about it. Thanks, Kenny > On Jan 1, 2021, at 2:35 PM, Mark Thompson wrote: > > Happy New Year! > > The fun part here is: > > AVMastering