Re: [FFmpeg-devel] [PATCH] avcodec: add setts bitstream filter

2021-02-13 Thread Gyan Doshi
Would be nice to be able to reference the current computed DTS/PTS. Also, FPS for video, and SR for audio. Regards, Gyan On 12-02-2021 03:26 pm, Paul B Mahol wrote: Will apply if nobody have comments. ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg

[FFmpeg-devel] [PATCH 1/2] avfilter/vf_gblur: factor out postscale function

2021-02-13 Thread Paul B Mahol
Signed-off-by: Paul B Mahol --- libavfilter/gblur.h| 2 ++ libavfilter/vf_gblur.c | 23 ++- 2 files changed, 16 insertions(+), 9 deletions(-) diff --git a/libavfilter/gblur.h b/libavfilter/gblur.h index 15a8167fe5..dce50671f6 100644 --- a/libavfilter/gblur.h +++ b/libavf

[FFmpeg-devel] [PATCH 2/2] avfilter/x86/vf_gblur: add postscale SIMD

2021-02-13 Thread Paul B Mahol
Signed-off-by: Paul B Mahol --- libavfilter/x86/vf_gblur.asm| 46 + libavfilter/x86/vf_gblur_init.c | 11 ++-- 2 files changed, 55 insertions(+), 2 deletions(-) diff --git a/libavfilter/x86/vf_gblur.asm b/libavfilter/x86/vf_gblur.asm index a25b1659f5..8fea

Re: [FFmpeg-devel] [PATCH] avcodec: add setts bitstream filter

2021-02-13 Thread Paul B Mahol
On Sat, Feb 13, 2021 at 10:16 AM Gyan Doshi wrote: > Would be nice to be able to reference the current computed DTS/PTS. > I doubt that would be useful. > > Also, FPS for video, and SR for audio. > FPS is not available, SR is. Patch welcome. > > Regards, > Gyan > > On 12-02-2021 03:26 pm, P

Re: [FFmpeg-devel] [PATCH] avcodec: add setts bitstream filter

2021-02-13 Thread Gyan Doshi
On 13-02-2021 05:11 pm, Paul B Mahol wrote: On Sat, Feb 13, 2021 at 10:16 AM Gyan Doshi wrote: Would be nice to be able to reference the current computed DTS/PTS. I doubt that would be useful. Since video streams may contain out-of-order packets, it would be useful to be able to peek at

Re: [FFmpeg-devel] [PATCH] avcodec: add setts bitstream filter

2021-02-13 Thread Paul B Mahol
On Sat, Feb 13, 2021 at 1:32 PM Gyan Doshi wrote: > > > On 13-02-2021 05:11 pm, Paul B Mahol wrote: > > On Sat, Feb 13, 2021 at 10:16 AM Gyan Doshi wrote: > > > >> Would be nice to be able to reference the current computed DTS/PTS. > >> > > I doubt that would be useful. > > Since video streams m

Re: [FFmpeg-devel] [PATCH] avcodec: add setts bitstream filter

2021-02-13 Thread Gyan Doshi
On 13-02-2021 06:05 pm, Paul B Mahol wrote: On Sat, Feb 13, 2021 at 1:32 PM Gyan Doshi wrote: On 13-02-2021 05:11 pm, Paul B Mahol wrote: On Sat, Feb 13, 2021 at 10:16 AM Gyan Doshi wrote: Would be nice to be able to reference the current computed DTS/PTS. I doubt that would be useful

Re: [FFmpeg-devel] [PATCH] avcodec: add setts bitstream filter

2021-02-13 Thread Paul B Mahol
On Sat, Feb 13, 2021 at 1:40 PM Gyan Doshi wrote: > > > On 13-02-2021 06:05 pm, Paul B Mahol wrote: > > On Sat, Feb 13, 2021 at 1:32 PM Gyan Doshi wrote: > > > >> > >> On 13-02-2021 05:11 pm, Paul B Mahol wrote: > >>> On Sat, Feb 13, 2021 at 10:16 AM Gyan Doshi wrote: > >>> > Would be nice

Re: [FFmpeg-devel] [PATCH v3] avcodec/mediacodecdec: Do not abort when H264/HEVC extradata extraction fails

2021-02-13 Thread Jan Ekström
On Sat, Feb 13, 2021 at 12:48 AM sfan5 wrote: > > 12.02.21 - 21:43 - Andreas Rheinhardt: > > sfan5: > >> Hi, > >> > >> > >> attached v2 patch after discussion on IRC with JEEB (as he already > >> mentioned). > >> > >> Only change is that the log level turns to debug when missing parameter > >> set

Re: [FFmpeg-devel] [PATCH] avformat/utils: force lowres to 0 in avformat_find_stream_info()

2021-02-13 Thread James Almer
On 2/6/2021 2:39 PM, James Almer wrote: Instead of applying it and then restoring the original codecpar dimensions. Signed-off-by: James Almer --- Alternative to "[PATCH] avformat/utils: always preserve container dimensions for all streams" while we figure out and decide how to properly make av

[FFmpeg-devel] [PATCH] avcodec: remove pointless lowres deprecation wrappers

2021-02-13 Thread James Almer
Neither the feature, public fields, or AVOptions were ever truly deprecated, and will also not be removed if this FF_API_ define was left in place, so remove it as it's misleading. Signed-off-by: James Almer --- libavcodec/avcodec.h | 13 - libavcodec/version.h | 3 --- libavformat/

Re: [FFmpeg-devel] [PATCH 1/2] avformat/mov: sanity check STSD entries

2021-02-13 Thread Michael Niedermayer
On Sun, Feb 07, 2021 at 09:50:02PM +0100, Michael Niedermayer wrote: > The choosen value is arbitrary. I am not sure if this is a good idea > but i dont immedeately see an alternative better way, it seems either > an arbitrary limit or OOM > > Fixes: OOM > Fixes: > 27492/clusterfuzz-testcase-mini

Re: [FFmpeg-devel] [PATCH 3/7] tools/target_dec_fuzzer: Adjust the threshold of vc1image

2021-02-13 Thread Michael Niedermayer
On Wed, Feb 03, 2021 at 12:13:30AM +0100, Michael Niedermayer wrote: > Fixes: Timeout > Fixes: > 30025/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_VC1IMAGE_fuzzer-5965511357759488 > > Found-by: continuous fuzzing process > https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg > Si

Re: [FFmpeg-devel] [PATCH 4/7] avcodec/fits: Check gcount and pcount being non negative

2021-02-13 Thread Michael Niedermayer
On Wed, Feb 03, 2021 at 12:13:31AM +0100, Michael Niedermayer wrote: > Fixes: signed integer overflow: 9223372036854775807 - -30069403896 cannot be > represented in type 'long' > Fixes: > 30046/clusterfuzz-testcase-minimized-ffmpeg_dem_FITS_fuzzer-5807144773484544 > > Found-by: continuous fuzzin

Re: [FFmpeg-devel] [PATCH 7/7] tools/target_dec_fuzzer: Adjust threshold for VMNC

2021-02-13 Thread Michael Niedermayer
On Wed, Feb 03, 2021 at 12:13:34AM +0100, Michael Niedermayer wrote: > Fixes: Timeout > Fixes: > 30055/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_VMNC_fuzzer-4739482576355328 > > Found-by: continuous fuzzing process > https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg > Signed

Re: [FFmpeg-devel] [PATCH 6/7] avcodec/fmvc: avoid copying uninitialized data

2021-02-13 Thread Michael Niedermayer
On Wed, Feb 03, 2021 at 12:13:33AM +0100, Michael Niedermayer wrote: > Fixes: Timeout > Fixes: > 30049/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_FMVC_fuzzer-5986909455253504 > > Found-by: continuous fuzzing process > https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg > Signed

Re: [FFmpeg-devel] [PATCH v2] Add support for the new key & value API in libaom.

2021-02-13 Thread Jan Ekström
On Wed, Feb 10, 2021 at 9:16 AM Bohan Li wrote: > > Hi Jan, > > Yes the modified patch looks good to me. > Please let me know if there is anything needed from my end. > > Thank you very much! > > Bohan > FYI, apparently the symbol is not exported in shared builds of libaom. I did test that versio

Re: [FFmpeg-devel] [PATCH 2/2] avfilter/x86/vf_gblur: add postscale SIMD

2021-02-13 Thread James Almer
On 2/13/2021 8:10 AM, Paul B Mahol wrote: Signed-off-by: Paul B Mahol --- libavfilter/x86/vf_gblur.asm| 46 + libavfilter/x86/vf_gblur_init.c | 11 ++-- 2 files changed, 55 insertions(+), 2 deletions(-) diff --git a/libavfilter/x86/vf_gblur.asm b/liba

[FFmpeg-devel] [PATCH] libavdevice/avfoundation: add buffer fifo and output packets in order they arrive

2021-02-13 Thread mindmark
From: Mark Reid Hi, This patch fixes audio issues I've had with some capture devices. The audio gets really choppy and stops working. This seems to be because avf_read_packet stops outputting the audio frames because a video frame happens to be available first. It base on the approach used in a

Re: [FFmpeg-devel] [PATCH v2] avformat/mov: fix timecode with counter mode flag set

2021-02-13 Thread Mark Reid
On Sun, Jan 31, 2021 at 1:27 AM Mark Reid wrote: > > > On Sat, Jan 16, 2021 at 5:48 PM wrote: > >> From: Mark Reid >> >> The current behaviour ends up squaring the avg_frame_rate if the conter >> mode flag is set. >> This messes up the timecode calculation, and looks to me as a regression >> th

[FFmpeg-devel] [PATCH] avcodec/movtextenc: fix compile warning for type-limits

2021-02-13 Thread Nuo Mi
CC libavcodec/mpegaudiodec_common.o libavcodec/movtextenc.c: In function ‘mov_text_style_start’: libavcodec/movtextenc.c:358:26: warning: comparison is always false due to limited range of data type [-Wtype-limits] 358 | if (s->count + 1 > SIZE_MAX / sizeof(*s->style_attributes) ||

[FFmpeg-devel] [PATCH] avutils/video_enc_params: remove unnecessary check to fix compile warning

2021-02-13 Thread Nuo Mi
This will fix following compile warning: libavutil/video_enc_params.c: In function ‘av_video_enc_params_alloc’: libavutil/video_enc