[FFmpeg-devel] [PATCH] avformat/hls_sample_encryption: Fix precedence

2021-11-07 Thread Andreas Rheinhardt
Fixes Coverity ticket #1492869. Signed-off-by: Andreas Rheinhardt --- libavformat/hls_sample_encryption.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavformat/hls_sample_encryption.c b/libavformat/hls_sample_encryption.c index 396fe97921..38795c7fb0 100644 --- a/libav

[FFmpeg-devel] [PATCH] avcodec/speexdec: Remove dead code

2021-11-07 Thread Andreas Rheinhardt
Fixes Coverity issue #1492840. Signed-off-by: Andreas Rheinhardt --- libavcodec/speexdec.c | 7 +-- 1 file changed, 1 insertion(+), 6 deletions(-) diff --git a/libavcodec/speexdec.c b/libavcodec/speexdec.c index 4c50f54f27..90e95f0785 100644 --- a/libavcodec/speexdec.c +++ b/libavcodec/spee

Re: [FFmpeg-devel] [PATCH] swscale/input: clamp rgbf32 values between 0, 1 before scaling

2021-11-07 Thread Mark Reid
On Sat, Nov 6, 2021 at 10:52 PM wrote: > From: Mark Reid > > if the float pixel * 65535.0f > 2147483647.0f > lrintf may overfow and return negative values, depending on implementation. > nan and +/-inf values may also be implementation defined > > clamp the values between 0,1 before scaling, so

Re: [FFmpeg-devel] [PATCH] configure: add initial RISC-V support

2021-11-07 Thread Anton Khirnov
Quoting Michael Niedermayer (2021-11-02 16:32:28) > On Mon, Nov 01, 2021 at 10:02:50PM -0400, Brad Smith wrote: > > On 11/1/2021 8:31 AM, Michael Niedermayer wrote: > > > > > On Fri, Oct 29, 2021 at 04:43:16PM -0400, Brad Smith wrote: > > > > On 10/29/2021 2:39 PM, Michael Niedermayer wrote: > > >

[FFmpeg-devel] [PATCH] avformat/matroskadec: Don't unnecessarily reduce aspect ratio

2021-11-07 Thread Andreas Rheinhardt
Fixes ticket #9497. Signed-off-by: Andreas Rheinhardt --- Will apply later tonight unless there are objections. libavformat/matroskadec.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavformat/matroskadec.c b/libavformat/matroskadec.c index b2c4927e43..a4bbbe954e 100644

[FFmpeg-devel] [PATCH] avformat/apngenc: Use UINT16_MAX instead of USHRT_MAX

2021-11-07 Thread Andreas Rheinhardt
The latter needn't be 16 bits. Signed-off-by: Andreas Rheinhardt --- Will apply this tonight unless there are objections. libavformat/apngenc.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/libavformat/apngenc.c b/libavformat/apngenc.c index c97c475274..767074ec

Re: [FFmpeg-devel] [PATCH] avformat/mxf: support MCA audio information

2021-11-07 Thread Marton Balint
On Fri, 5 Nov 2021, Marc-Antoine Arnaud wrote: --- libavformat/mxf.h| 1 + libavformat/mxfdec.c | 280 ++- 2 files changed, 275 insertions(+), 6 deletions(-) Could you mention in the commit description which standards/recommendations cover MCA i

[FFmpeg-devel] [PATCH] avcodec/libx264: Use av_memdup() where appropriate

2021-11-07 Thread Andreas Rheinhardt
Signed-off-by: Andreas Rheinhardt --- Will apply tonight unless there are objections. libavcodec/libx264.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/libavcodec/libx264.c b/libavcodec/libx264.c index 0766b4a950..29546ebf06 100644 --- a/libavcodec/libx264.c +++ b/libavc

Re: [FFmpeg-devel] [PATCH] avcodec/libx264: Use av_memdup() where appropriate

2021-11-07 Thread James Almer
On 11/7/2021 8:33 AM, Andreas Rheinhardt wrote: Signed-off-by: Andreas Rheinhardt --- Will apply tonight unless there are objections. libavcodec/libx264.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/libavcodec/libx264.c b/libavcodec/libx264.c index 0766b4a950..29546

[FFmpeg-devel] [PATCH] avformat/mov: Check channels for mov_parse_stsd_audio()

2021-11-07 Thread Michael Niedermayer
Fixes: signed integer overflow: -776522110086937600 * 16 cannot be represented in type 'long' Fixes: 40563/clusterfuzz-testcase-minimized-ffmpeg_dem_MOV_fuzzer-6644829447127040 Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg Signed-off-by: Mic

Re: [FFmpeg-devel] [PATCH] avformat/matroskadec: Don't unnecessarily reduce aspect ratio

2021-11-07 Thread James Almer
On 11/7/2021 7:42 AM, Andreas Rheinhardt wrote: Fixes ticket #9497. Signed-off-by: Andreas Rheinhardt --- Will apply later tonight unless there are objections. libavformat/matroskadec.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavformat/matroskadec.c b/libavform

Re: [FFmpeg-devel] [PATCH] http: make caching of redirect url optional

2021-11-07 Thread Marton Balint
On Tue, 2 Nov 2021, Eran Kornblau wrote: Hi all, The attached patch makes the default behavior of caching HTTP redirects optional. Is caching a redirected URL allowed per the HTTP spec? If not, then no caching should take place, or the caching should be optional, and not the default.

[FFmpeg-devel] [PATCH 1/3] avcodec/libx264: Check for overflow if necessary

2021-11-07 Thread Andreas Rheinhardt
Signed-off-by: Andreas Rheinhardt --- libavcodec/libx264.c | 14 -- 1 file changed, 12 insertions(+), 2 deletions(-) diff --git a/libavcodec/libx264.c b/libavcodec/libx264.c index 0766b4a950..8711c72131 100644 --- a/libavcodec/libx264.c +++ b/libavcodec/libx264.c @@ -138,13 +138,23 @

[FFmpeg-devel] [PATCH 2/3] avcodec/libx264: Remove always-false checks

2021-11-07 Thread Andreas Rheinhardt
Always false since this encoder was switched to encode2 and ff_alloc_packet() in 06484d0b8a7d4d1a694ba7ab277e2ec32d6558d7 and f2b20b7a8b6fcbcd8cc669f5211e4e2ed7d8e9f3. Signed-off-by: Andreas Rheinhardt --- libavcodec/libx264.c | 6 +- 1 file changed, 1 insertion(+), 5 deletions(-) diff --gi

[FFmpeg-devel] [PATCH 3/3] avcodec/libx264: Simplify copying packet data

2021-11-07 Thread Andreas Rheinhardt
x264.h: "the payloads of all output NALs are guaranteed to be sequential in memory." Therefore we can omit the loop. Signed-off-by: Andreas Rheinhardt --- libavcodec/libx264.c | 9 - 1 file changed, 4 insertions(+), 5 deletions(-) diff --git a/libavcodec/libx264.c b/libavcodec/libx264.c

[FFmpeg-devel] [PATCH]{WIP} avfilter: add moddif video filter

2021-11-07 Thread Paul B Mahol
Signed-off-by: Paul B Mahol --- Allows combination of output of spatial only deinterlacers and spatio-temporal deinterlacers. Gives overall higher PSNR result. Spatial only deinterlacer output is used in case spatio-temporal one would use simple spatial interpolations. --- libavfilter/Makefi

Re: [FFmpeg-devel] [PATCH 3/3] avcodec/libx264: Simplify copying packet data

2021-11-07 Thread James Almer
On 11/7/2021 11:36 AM, Andreas Rheinhardt wrote: x264.h: "the payloads of all output NALs are guaranteed to be sequential in memory." Therefore we can omit the loop. Signed-off-by: Andreas Rheinhardt --- libavcodec/libx264.c | 9 - 1 file changed, 4 insertions(+), 5 deletions(-) dif

Re: [FFmpeg-devel] [PATCH] http: make caching of redirect url optional

2021-11-07 Thread Hendrik Leppkes
On Sun, Nov 7, 2021 at 3:33 PM Marton Balint wrote: > > > > On Tue, 2 Nov 2021, Eran Kornblau wrote: > > > Hi all, > > > > The attached patch makes the default behavior of caching HTTP redirects > > optional. > > Is caching a redirected URL allowed per the HTTP spec? If not, then no > caching sho

[FFmpeg-devel] [PATCH] avformat/demux: allow total size of packets in raw_packet_buffer to reach probesize

2021-11-07 Thread Marton Balint
Previously this was hardcoded to 250 bytes, so probing of the stream codecs was always limited by this, and not probesize. Also keep track of the actual size of packets in raw_packet_buffer and not the remaining size for simplicity. Fixes ticket #5860. Signed-off-by: Marton Balint --- liba

Re: [FFmpeg-devel] [PATCH 3/5] avformat/imf: Demuxer implementation

2021-11-07 Thread Pierre-Anthony Lemieux
The comments have been addressed in v3 of the patch. On Sun, Oct 31, 2021 at 1:22 PM Pierre-Anthony Lemieux wrote: > > On Sun, Oct 31, 2021 at 11:45 AM James Almer wrote: > > > > On 10/31/2021 3:34 PM, Pierre-Anthony Lemieux wrote: > > >>> The functions are not static and are defined in imf_in

Re: [FFmpeg-devel] [PATCH] avfilter/af_apad: do not add infinte silence for zero pad_dur or whole_dur

2021-11-07 Thread Marton Balint
On Thu, 29 Jul 2021, Marton Balint wrote: Unfortunately pad_len and pad_dur behaviour was different if 0 was specified, pad_dur handled 0 duration as infinity, for pad_len, infinity was -1. Let's make the behaviour consistent by handling 0 duration for pad_dur and whole_dur as indeed 0 durat

Re: [FFmpeg-devel] [PATCH v3] libavfilter: add a gblur_vulkan filter

2021-11-07 Thread Chen, Wenbin
> 9 Sept 2021, 07:44 by jianhua...@intel.com: > > > This commit adds a powerful and customizable gblur Vulkan filter, > > which provides a maximum 127x127 kernel size of Gaussian Filter. > > The size could be adjusted by requirements on quality or performance. > > > > The following command is on h

Re: [FFmpeg-devel] [PATCH] avformat/hls_sample_encryption: Fix precedence

2021-11-07 Thread Steven Liu
Andreas Rheinhardt 于2021年11月7日周日 下午3:57写道: > > Fixes Coverity ticket #1492869. > > Signed-off-by: Andreas Rheinhardt > --- > libavformat/hls_sample_encryption.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/libavformat/hls_sample_encryption.c > b/libavformat/hls_sampl

[FFmpeg-devel] [PATCH v2 2/2] swscale/input: clamp rgbf32 values between 0, 1 before scaling

2021-11-07 Thread mindmark
From: Mark Reid if the float pixel * 65535.0f > 2147483647.0f lrintf may overfow and return negative values, depending on implementation. nan and +/-inf values may also be implementation defined clamp the values between 0,1 before scaling, so lrintf always works. values <=0.0f, -inf, nan = 0.0f

[FFmpeg-devel] [PATCH v2 1/2] swscale/input: unify grayf32 funcs with rgbf32 funcs

2021-11-07 Thread mindmark
From: Mark Reid --- libswscale/input.c | 36 +++- 1 file changed, 11 insertions(+), 25 deletions(-) diff --git a/libswscale/input.c b/libswscale/input.c index 336f957c8c..90efdd2ffc 100644 --- a/libswscale/input.c +++ b/libswscale/input.c @@ -1013,31 +1013,19 @@

Re: [FFmpeg-devel] [PATCH] http: make caching of redirect url optional

2021-11-07 Thread Eran Kornblau
> > On Tue, 2 Nov 2021, Eran Kornblau wrote: > > > Hi all, > > > > The attached patch makes the default behavior of caching HTTP redirects > > optional. > > Is caching a redirected URL allowed per the HTTP spec? If not, then no > caching should take place, or the caching should be optional, an

[FFmpeg-devel] [PATCH] avformat/mpegtsenc: Implement muxing SCTE-35 and EPG packets

2021-11-07 Thread Maksym Veremeyenko
Hi, Attached patch implement muxing SCTE-35 and EPG packets into mpegts stream. -- Maksym Veremeyenko From dc1b87f5e34c688bb3691767f8026a2815aab03e Mon Sep 17 00:00:00 2001 From: Maksym Veremeyenko Date: Mon, 8 Nov 2021 08:10:59 +0200 Subject: [PATCH] avformat/mpegtsenc: Implement muxing SCTE-