Re: [FFmpeg-devel] [PATCH 1/2] avcodec/speexdec: Check frames_per_packet more completely

2021-10-14 Thread Paul B Mahol
On Fri, Oct 15, 2021 at 12:20 AM James Almer wrote: > On 10/14/2021 7:13 PM, Michael Niedermayer wrote: > > Fixes: signed integer overflow: 2105344 * 539033345 cannot be > represented in type 'int' > > Fixes: out of array write > > Fixes: > 39956/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_

Re: [FFmpeg-devel] [PATCH] doc/filters: added possible values to @item win_func

2021-10-14 Thread Gyan Doshi
On 2021-10-15 12:35 am, Paul B Mahol wrote: will revert in next 96 hours. Devote time to something productive. In any case, not all filters have identical lists. spectrumsynth supports limited functions. many don't recognize the string 'hann'. The defaults also vary. Harmonize those first

Re: [FFmpeg-devel] [PATCH v5 1/9] cbs_av1: fix incorrect data type

2021-10-14 Thread Wang, Fei W
On Tue, 2021-10-12 at 16:23 +0800, Fei Wang wrote: > Since order_hint_bits_minus_1 range is 0~7, cur_frame_hint can be > most 128. And similar return value for cbs_av1_get_relative_dist. > So if plus them and use int8_t for the result may lose its precision. > > Signed-off-by: Fei Wang > --- > up

Re: [FFmpeg-devel] [PATCH 8/8] avformat/mpegts: return proper error codes

2021-10-14 Thread lance . lmwang
On Thu, Oct 14, 2021 at 09:46:37PM +0200, Marton Balint wrote: > > > On Thu, 14 Oct 2021, lance.lmw...@gmail.com wrote: > > > From: Limin Wang > > > > Signed-off-by: Limin Wang > > --- > > libavformat/mpegts.c | 12 ++-- > > 1 file changed, 6 insertions(+), 6 deletions(-) > > No, pars

Re: [FFmpeg-devel] [PATCH 1/2] avcodec/speexdec: Check frames_per_packet more completely

2021-10-14 Thread James Almer
On 10/14/2021 7:13 PM, Michael Niedermayer wrote: Fixes: signed integer overflow: 2105344 * 539033345 cannot be represented in type 'int' Fixes: out of array write Fixes: 39956/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_SPEEX_fuzzer-4766419250708480 Found-by: continuous fuzzing process

[FFmpeg-devel] [PATCH 2/2] avcodec/ttadsp: Fix integer overflows in tta_filter_process_c()

2021-10-14 Thread Michael Niedermayer
Fixes: signed integer overflow: 822841647 + 1647055738 cannot be represented in type 'int' Fixes: 39935/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_TTA_fuzzer-4592657142251520 Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg Signed-off-by

[FFmpeg-devel] [PATCH 1/2] avcodec/speexdec: Check frames_per_packet more completely

2021-10-14 Thread Michael Niedermayer
Fixes: signed integer overflow: 2105344 * 539033345 cannot be represented in type 'int' Fixes: out of array write Fixes: 39956/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_SPEEX_fuzzer-4766419250708480 Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/proj

Re: [FFmpeg-devel] [PATCH 8/8] avformat/mpegts: return proper error codes

2021-10-14 Thread Marton Balint
On Thu, 14 Oct 2021, lance.lmw...@gmail.com wrote: From: Limin Wang Signed-off-by: Limin Wang --- libavformat/mpegts.c | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) No, parse_stream_identifier_desc intentionally returns -1 for error, check how the code works which us

Re: [FFmpeg-devel] [PATCH] doc/filters: added possible values to @item win_func

2021-10-14 Thread Paul B Mahol
will revert in next 96 hours. ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".

Re: [FFmpeg-devel] [PATCH 7/8] avformat/mpegts: Fix for the DOVI video stream descriptor

2021-10-14 Thread lance . lmwang
On Thu, Oct 14, 2021 at 10:12:05AM -0400, quietvoid wrote: > On Thu, Oct 14, 2021 at 9:52 AM wrote: > > > On Thu, Oct 14, 2021 at 09:45:45AM -0400, quietvoid wrote: > > > On Thu, Oct 14, 2021 at 9:36 AM wrote: > > > > > > > On Thu, Oct 14, 2021 at 09:18:16AM -0400, f tcChlisop0 wrote: > > > > >

Re: [FFmpeg-devel] [PATCH 7/8] avformat/mpegts: Fix for the DOVI video stream descriptor

2021-10-14 Thread quietvoid
On Thu, Oct 14, 2021 at 9:52 AM wrote: > On Thu, Oct 14, 2021 at 09:45:45AM -0400, quietvoid wrote: > > On Thu, Oct 14, 2021 at 9:36 AM wrote: > > > > > On Thu, Oct 14, 2021 at 09:18:16AM -0400, f tcChlisop0 wrote: > > > > On Thu, Oct 14, 2021 at 9:10 AM wrote: > > > > > > > > > From: Limin Wan

Re: [FFmpeg-devel] [PATCH 7/8] avformat/mpegts: Fix for the DOVI video stream descriptor

2021-10-14 Thread lance . lmwang
On Thu, Oct 14, 2021 at 09:45:45AM -0400, quietvoid wrote: > On Thu, Oct 14, 2021 at 9:36 AM wrote: > > > On Thu, Oct 14, 2021 at 09:18:16AM -0400, f tcChlisop0 wrote: > > > On Thu, Oct 14, 2021 at 9:10 AM wrote: > > > > > > > From: Limin Wang > > > > > > > > By < > v1.2>> > > > > > > > > Signe

Re: [FFmpeg-devel] [PATCH 7/8] avformat/mpegts: Fix for the DOVI video stream descriptor

2021-10-14 Thread quietvoid
On Thu, Oct 14, 2021 at 9:36 AM wrote: > On Thu, Oct 14, 2021 at 09:18:16AM -0400, f tcChlisop0 wrote: > > On Thu, Oct 14, 2021 at 9:10 AM wrote: > > > > > From: Limin Wang > > > > > > By < v1.2>> > > > > > > Signed-off-by: Limin Wang > > > --- > > > libavformat/mpegts.c | 11 +-- > >

Re: [FFmpeg-devel] [PATCH 7/8] avformat/mpegts: Fix for the DOVI video stream descriptor

2021-10-14 Thread lance . lmwang
On Thu, Oct 14, 2021 at 09:18:16AM -0400, f tcChlisop0 wrote: > On Thu, Oct 14, 2021 at 9:10 AM wrote: > > > From: Limin Wang > > > > By <> > > > > Signed-off-by: Limin Wang > > --- > > libavformat/mpegts.c | 11 +-- > > 1 file changed, 9 insertions(+), 2 deletions(-) > > > > diff --gi

Re: [FFmpeg-devel] [PATCH 7/8] avformat/mpegts: Fix for the DOVI video stream descriptor

2021-10-14 Thread f tcChlisop0
On Thu, Oct 14, 2021 at 9:10 AM wrote: > From: Limin Wang > > By <> > > Signed-off-by: Limin Wang > --- > libavformat/mpegts.c | 11 +-- > 1 file changed, 9 insertions(+), 2 deletions(-) > > diff --git a/libavformat/mpegts.c b/libavformat/mpegts.c > index 44d9298..774964d 100644 > ---

[FFmpeg-devel] [PATCH 8/8] avformat/mpegts: return proper error codes

2021-10-14 Thread lance . lmwang
From: Limin Wang Signed-off-by: Limin Wang --- libavformat/mpegts.c | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/libavformat/mpegts.c b/libavformat/mpegts.c index 774964d..3157e3a 100644 --- a/libavformat/mpegts.c +++ b/libavformat/mpegts.c @@ -2270,22 +2270,

[FFmpeg-devel] [PATCH 7/8] avformat/mpegts: Fix for the DOVI video stream descriptor

2021-10-14 Thread lance . lmwang
From: Limin Wang By <> Signed-off-by: Limin Wang --- libavformat/mpegts.c | 11 +-- 1 file changed, 9 insertions(+), 2 deletions(-) diff --git a/libavformat/mpegts.c b/libavformat/mpegts.c index 44d9298..774964d 100644 --- a/libavformat/mpegts.c +++ b/libavformat/mpegts.c @@ -2178,6 +

[FFmpeg-devel] [PATCH 6/8] avformat/movenc: support dvwC box for Dolby Vision

2021-10-14 Thread lance . lmwang
From: Limin Wang By <> Signed-off-by: Limin Wang --- libavformat/movenc.c | 6 -- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/libavformat/movenc.c b/libavformat/movenc.c index d43a086..37d4403 100644 --- a/libavformat/movenc.c +++ b/libavformat/movenc.c @@ -1912,7 +1912,9

[FFmpeg-devel] [PATCH 5/8] avformat/mov: support dvwC box for Dolby Vision

2021-10-14 Thread lance . lmwang
From: Limin Wang By <> Signed-off-by: Limin Wang --- libavformat/mov.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/libavformat/mov.c b/libavformat/mov.c index 57c67e3..49a8c27 100644 --- a/libavformat/mov.c +++ b/libavformat/mov.c @@ -7074,7 +7074,7 @@ static int mov_

[FFmpeg-devel] [PATCH 4/8] avcodec/atsc_a53: use AVERROR_INVALIDDATA

2021-10-14 Thread lance . lmwang
From: Limin Wang Signed-off-by: Limin Wang --- libavcodec/atsc_a53.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/libavcodec/atsc_a53.c b/libavcodec/atsc_a53.c index 2d89ef5..29ec71b 100644 --- a/libavcodec/atsc_a53.c +++ b/libavcodec/atsc_a53.c @@ -73,7 +73,7 @@ in

[FFmpeg-devel] [PATCH 3/8] avcodec/hevc_sei: remove the duplicate check

2021-10-14 Thread lance . lmwang
From: Limin Wang Signed-off-by: Limin Wang --- libavcodec/hevc_sei.c | 3 --- 1 file changed, 3 deletions(-) diff --git a/libavcodec/hevc_sei.c b/libavcodec/hevc_sei.c index 29d0346..7fd8226 100644 --- a/libavcodec/hevc_sei.c +++ b/libavcodec/hevc_sei.c @@ -181,9 +181,6 @@ static int decode_r

[FFmpeg-devel] [PATCH 2/8] avfilter/vf_showinfo: small adjustments for the HDR10+ dump

2021-10-14 Thread lance . lmwang
From: Limin Wang summary for the adjustments: 1, remove the extra "," in the ,} ...{0.2004,0.3001,0.4008,0.5005,0.6002,0.7009,0.8006,0.9013,} to ...{0.2004,0.3001,0.4008,0.5005,0.6002,0.7009,0.8006,0.9013} 2, add "," between the } and new field } fraction_bright_pixels to }, fraction_bright_pixe

[FFmpeg-devel] [PATCH 1/8] avfilter/af_replaygain: use fabsf() instead of fabs()

2021-10-14 Thread lance . lmwang
From: Limin Wang Signed-off-by: Limin Wang --- libavfilter/af_replaygain.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/libavfilter/af_replaygain.c b/libavfilter/af_replaygain.c index 4bf2763..da5c196 100644 --- a/libavfilter/af_replaygain.c +++ b/libavfilter/af_repla

Re: [FFmpeg-devel] [PATCH v2 1/3] avformat/argo_asf: cleanup and NULL-terminate name field in header

2021-10-14 Thread Zane van Iperen
On 14/10/21 10:45 pm, "zhilizhao(赵志立)" wrote: } ArgoASFFileHeader; I failed to see why null-terminator is needed from the commit message or from the source code. See https://ffmpeg.org/pipermail/ffmpeg-devel/2021-October/286939.html Sorry for the noise, email client doesn’t show the patc

Re: [FFmpeg-devel] [PATCH v2 1/3] avformat/argo_asf: cleanup and NULL-terminate name field in header

2021-10-14 Thread zhilizhao(赵志立)
> On Oct 14, 2021, at 8:25 PM, Zane van Iperen wrote: > > > > On 14/10/21 10:11 pm, "zhilizhao(赵志立)" wrote: > >>> typedef struct ArgoASFFileHeader { >>> uint32_tmagic; /*< Magic Number, {'A', 'S', 'F', '\0'} */ >>> @@ -40,7 +41,7 @@ typedef struct ArgoASFFileHeader { >>>

Re: [FFmpeg-devel] [PATCH v2 1/3] avformat/argo_asf: cleanup and NULL-terminate name field in header

2021-10-14 Thread Zane van Iperen
On 14/10/21 10:11 pm, "zhilizhao(赵志立)" wrote: typedef struct ArgoASFFileHeader { uint32_tmagic; /*< Magic Number, {'A', 'S', 'F', '\0'} */ @@ -40,7 +41,7 @@ typedef struct ArgoASFFileHeader { uint16_tversion_minor; /*< File Minor Version. */ uint32_tnum_chu

Re: [FFmpeg-devel] [PATCH v2 1/3] avformat/argo_asf: cleanup and NULL-terminate name field in header

2021-10-14 Thread zhilizhao(赵志立)
> On Oct 12, 2021, at 7:02 PM, Zane van Iperen wrote: > > Signed-off-by: Zane van Iperen > --- > libavformat/argo_asf.c | 8 > libavformat/argo_asf.h | 3 ++- > 2 files changed, 6 insertions(+), 5 deletions(-) > > diff --git a/libavformat/argo_asf.c b/libavformat/argo_asf.c > index 7e

Re: [FFmpeg-devel] [PATCH v2 1/3] avformat/argo_asf: cleanup and NULL-terminate name field in header

2021-10-14 Thread Zane van Iperen
Will apply tomorrow if no objections. Zane On 12/10/21 9:02 pm, Zane van Iperen wrote: Signed-off-by: Zane van Iperen --- libavformat/argo_asf.c | 8 libavformat/argo_asf.h | 3 ++- 2 files changed, 6 insertions(+), 5 deletions(-) diff --git a/libavformat/argo_asf.c b/libavformat

[FFmpeg-devel] [share a stupid and fun patch] avformat: add a fileptr protocol

2021-10-14 Thread zhilizhao(赵志立)
Just share a patch, not meant for merge. Basically, the patch adds a new protocol which looks like fd:// or pipe:, but deals with FILE *. You may ask why? It’s useless if the FILE *ptr is a regular one, unless combined with FILE * open_memstream(char **bufp, size_t *sizep); FILE

Re: [FFmpeg-devel] [PATCH] avfilter/vf_fftfilt: Use av_clip_uintp2

2021-10-14 Thread Paul B Mahol
or av_clip_uint8(), whatever you prefer. ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".

Re: [FFmpeg-devel] [PATCH v2 1/4] libswscale/x86/rgb2rgb: add shuffle_bytes avx2

2021-10-14 Thread Paul B Mahol
looks very trivial, so will apply shortly ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".

Re: [FFmpeg-devel] [PATCH v3 1/4] libswscale/x86/rgb2rgb: add shuffle_bytes avx2

2021-10-14 Thread Wu, Jianhua
Ping. > -Original Message- > From: Wu, Jianhua > Sent: Thursday, September 30, 2021 4:44 PM > To: ffmpeg-devel@ffmpeg.org > Cc: Wu, Jianhua > Subject: [PATCH v3 1/4] libswscale/x86/rgb2rgb: add shuffle_bytes avx2 > > Performance data(Less is better): > shuffle_bytes_ssse3 3.64654