Re: [FFmpeg-devel] Request for review - x265 User Data Unregistered SEI patch

2021-07-11 Thread Derek Buitenhuis
Hi Brad, On 7/8/2021 4:31 AM, Brad Hards wrote: > About a month ago, I submitted a patch to add User Data Unregistered SEI > writing to the x265 implementation. > > See http://ffmpeg.org/pipermail/ffmpeg-devel/2021-June/280978.html[1] > and > https://patchwork.ffmpeg.org/project/ffmpeg/patch/202

Re: [FFmpeg-devel] [PATCH] avcodec/libdav1d: parse sequence headers in extradata if available

2021-07-11 Thread Derek Buitenhuis
On 7/9/2021 3:53 PM, James Almer wrote: > +res = dav1d_parse_sequence_header(&seq, c->extradata + offset, > + c->extradata_size - offset); > +if (res < 0) { > +av_log(c, explode ? AV_LOG_ERROR : AV_LOG_INFO, > + "Error decoding ext

Re: [FFmpeg-devel] [PATCH V2 3/6] lavfi/dnn_backend_tf: Request-based Execution

2021-07-11 Thread Guo, Yejun
> -Original Message- > From: ffmpeg-devel On Behalf Of > Shubhanshu Saxena > Sent: 2021年7月5日 18:31 > To: ffmpeg-devel@ffmpeg.org > Cc: Shubhanshu Saxena > Subject: [FFmpeg-devel] [PATCH V2 3/6] lavfi/dnn_backend_tf: Request- > based Execution > > This commit uses TFRequestItem and the

Re: [FFmpeg-devel] [PATCH] avcodec/libdav1d: parse sequence headers in extradata if available

2021-07-11 Thread James Almer
On 7/11/2021 8:40 AM, Derek Buitenhuis wrote: On 7/9/2021 3:53 PM, James Almer wrote: +res = dav1d_parse_sequence_header(&seq, c->extradata + offset, + c->extradata_size - offset); +if (res < 0) { +av_log(c, explode ? AV_LOG_ERROR : AV_LOG_IN

Re: [FFmpeg-devel] Request for review - x265 User Data Unregistered SEI patch

2021-07-11 Thread Timo Rothenpieler
On 11.07.2021 14:01, Derek Buitenhuis wrote: Hi Brad, On 7/8/2021 4:31 AM, Brad Hards wrote: About a month ago, I submitted a patch to add User Data Unregistered SEI writing to the x265 implementation. See http://ffmpeg.org/pipermail/ffmpeg-devel/2021-June/280978.html[1] and https://patchwork.

[FFmpeg-devel] [PATCH 1/3] avcodec/exr: Check uncompressed_size against max_pixels

2021-07-11 Thread Michael Niedermayer
Fixes: Timeout Fixes: 35286/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_EXR_fuzzer-6557139802914816 Fixes: 31253/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_EXR_fuzzer-4901782326214656 Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/projects/ffmpe

[FFmpeg-devel] [PATCH 2/3] avformat/mxfdec: Check size for shrinking

2021-07-11 Thread Michael Niedermayer
av_shrink_packet() takes int size, so size must fit in int Fixes: out of array access Fixes: 35607/clusterfuzz-testcase-minimized-ffmpeg_dem_MXF_fuzzer-4875541323841536 Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg Signed-off-by: Michael Nied

[FFmpeg-devel] [PATCH 3/3] avformat/mov: do not ignore errors in mov_metadata_hmmt()

2021-07-11 Thread Michael Niedermayer
Fixes: Timeout Fixes: 35637/clusterfuzz-testcase-minimized-ffmpeg_dem_MOV_fuzzer-6311060272447488 Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg Signed-off-by: Michael Niedermayer --- libavformat/mov.c | 2 ++ 1 file changed, 2 insertions(+)

Re: [FFmpeg-devel] [PATCH V2 3/6] lavfi/dnn_backend_tf: Request-based Execution

2021-07-11 Thread Shubhanshu Saxena
On Sun, Jul 11, 2021 at 6:25 PM Guo, Yejun wrote: > > > > -Original Message- > > From: ffmpeg-devel On Behalf Of > > Shubhanshu Saxena > > Sent: 2021年7月5日 18:31 > > To: ffmpeg-devel@ffmpeg.org > > Cc: Shubhanshu Saxena > > Subject: [FFmpeg-devel] [PATCH V2 3/6] lavfi/dnn_backend_tf: Req

[FFmpeg-devel] [PATCH 1/2] lavfi/dnn_backend_openvino.c: Fix Memory Leak in execute_model_ov

2021-07-11 Thread Shubhanshu Saxena
In cases where the execution inside the function execute_model_ov fails, the OVRequestItem must be pushed back to the request_queue before returning the error. In case pushing back fails, release the allocated memory. Signed-off-by: Shubhanshu Saxena --- libavfilter/dnn/dnn_backend_openvino.c |

[FFmpeg-devel] [PATCH 2/2] lavfi/dnn_backend_ov: Rename RequestItem to OVRequestItem

2021-07-11 Thread Shubhanshu Saxena
Rename RequestItem to OVRequestItem in the OpenVINO backend to avoid confusion. Signed-off-by: Shubhanshu Saxena --- libavfilter/dnn/dnn_backend_openvino.c | 24 1 file changed, 12 insertions(+), 12 deletions(-) diff --git a/libavfilter/dnn/dnn_backend_openvino.c b/lib

Re: [FFmpeg-devel] [PATCH] cafenc: fill in avg. packet size later if unknown

2021-07-11 Thread ffmpegandmahanstreamer
On 2021-07-10 03:42, Lynne wrote: This doesn't move the pointer back to the file end if par->block_align is set. I think that's fine though, since the function writes the trailer, which should mean that nothing more needs to be written. Patch LGTM. But please, someone yell at Apple to support O

Re: [FFmpeg-devel] [PATCH] avfilter: add afwtdn filter

2021-07-11 Thread ffmpegandmahanstreamer
On 2021-07-10 15:20, Paul B Mahol wrote: Signed-off-by: Paul B Mahol --- doc/filters.texi | 60 ++ libavfilter/Makefile |1 + libavfilter/af_afwtdn.c | 1349 ++ libavfilter/allfilters.c |1 + 4 files changed, 1411 insertions(+) create

[FFmpeg-devel] [PATCH] mxfdec.c: fixed frame wrapping detection for MXFGCP1FrameWrappedPicture essence container

2021-07-11 Thread pal
From: Pierre-Anthony Lemieux Signed-off-by: Pierre-Anthony Lemieux --- Notes: For JPEG 2000 essence, the MXF input format module currently uses the value of byte 14 of the essence container UL to determines whether the J2K essence is clip- (byte 14 is 0x02) or frame-wrapped (byte 14 i

Re: [FFmpeg-devel] [PATCH] avcodec/qsvenc: clip global_quality for ICQ modes.

2021-07-11 Thread Xiang, Haihao
On Sun, 2021-07-11 at 10:14 +0530, Gyan Doshi wrote: > Allowed range is 1 to 51. > > Ref: > https://software.intel.com/content/www/us/en/develop/articles/advanced-bitrate-control-methods-in-intel-media-sdk.html > --- > doc/encoders.texi | 3 ++- > libavcodec/qsvenc.c | 2 +- > 2 files changed,

Re: [FFmpeg-devel] [PATCH] avcodec/qsvenc: clip global_quality for ICQ modes.

2021-07-11 Thread Gyan Doshi
On 2021-07-12 06:44, Xiang, Haihao wrote: On Sun, 2021-07-11 at 10:14 +0530, Gyan Doshi wrote: Allowed range is 1 to 51. Ref: https://software.intel.com/content/www/us/en/develop/articles/advanced-bitrate-control-methods-in-intel-media-sdk.html --- doc/encoders.texi | 3 ++- libavcodec/

[FFmpeg-devel] [PATCH V4] lavf/vf_ocr: add subregion support

2021-07-11 Thread Lingjiang Fang
follow comments from Steven Liu --- doc/filters.texi | 8 libavfilter/vf_ocr.c | 45 +++- 2 files changed, 52 insertions(+), 1 deletion(-) diff --git a/doc/filters.texi b/doc/filters.texi index d991c06628..f41ba0ce46 100644 --- a/doc/filters.t

[FFmpeg-devel] [PATCH V4] lavf/vf_ocr: add subregion support

2021-07-11 Thread Lingjiang Fang
follow comments from Steven Liu --- doc/filters.texi | 8 libavfilter/vf_ocr.c | 45 +++- 2 files changed, 52 insertions(+), 1 deletion(-) diff --git a/doc/filters.texi b/doc/filters.texi index d991c06628..f41ba0ce46 100644 --- a/doc/filters.t

[FFmpeg-devel] [PATCH V4] lavf/vf_ocr: add subregion support

2021-07-11 Thread Lingjiang Fang
follow comments from Steven Liu --- doc/filters.texi | 8 libavfilter/vf_ocr.c | 45 +++- 2 files changed, 52 insertions(+), 1 deletion(-) diff --git a/doc/filters.texi b/doc/filters.texi index d991c06628..f41ba0ce46 100644 --- a/doc/filters.t

Re: [FFmpeg-devel] [PATCH V4] lavf/vf_ocr: add subregion support

2021-07-11 Thread Lingjiang Fang
On Mon, 12 Jul 2021 13:09:26 +0800 Lingjiang Fang wrote: Sorry, I made a mistake when reply to previous email, ignore this please : ( > follow comments from Steven Liu > --- > doc/filters.texi | 8 > libavfilter/vf_ocr.c | 45 > +++- 2 files

Re: [FFmpeg-devel] [PATCH V4] lavf/vf_ocr: add subregion support

2021-07-11 Thread Lingjiang Fang
On Mon, 12 Jul 2021 13:11:28 +0800 Lingjiang Fang wrote: Sorry, I made a mistake when reply to previous email, ignore this please : ( > follow comments from Steven Liu > --- > doc/filters.texi | 8 > libavfilter/vf_ocr.c | 45 > +++- 2 files