[FFmpeg-devel] [PATCH] lavc/pngdec: actually use SIMD version for bpp<=2

2021-05-11 Thread Nicolas Frattaroli
The SIMD version was only used by the bpp > 2 case. This patch uses the SIMD version for both code paths. Change has been tested to still work with a 1-bpp image that uses paeth, to generate one: pngnq-s9 input.png optipng -f4 -force input-nq8.png --- libavcodec/pngdec.c | 2 +- 1 file changed,

Re: [FFmpeg-devel] [PATCH] avcodec/allcodecs: Don't include libx264 twice in the list of codecs

2021-05-11 Thread Andreas Rheinhardt
Andreas Rheinhardt: > Since a247ac640df3da573cd661065bf53f37863e2b46, allcodecs.c contained > two lines that matched the regex used by find_filters_extern in > configure; as a result, libx264 appeared twice the list of codecs > (if enabled). > > Fix this by using only one matching line by adding

Re: [FFmpeg-devel] [PATCH 1/5] lavfi/dnn_backend_native_layer_avgpool.h: Documentation

2021-05-11 Thread Guo, Yejun
> -Original Message- > From: ffmpeg-devel On Behalf Of > Shubhanshu Saxena > Sent: 2021年5月8日 20:10 > To: ffmpeg-devel@ffmpeg.org > Cc: Shubhanshu Saxena > Subject: [FFmpeg-devel] [PATCH 1/5] > lavfi/dnn_backend_native_layer_avgpool.h: Documentation > > Add documentation for Average

Re: [FFmpeg-devel] [PATCH] lavf/flvdec: normalize exporting date metadata

2021-05-11 Thread Alexander Strasser
On 2021-05-11 17:51 +0200, Anton Khirnov wrote: > Quoting Alexander Strasser (2021-05-10 15:35:02) > > On 2021-05-10 10:22 +0200, Anton Khirnov wrote: > > > Export them in UTC, not the local timezone. This way the output is > > > the same everywhere. The timezone information stored in the file is

Re: [FFmpeg-devel] [PATCH] avformat/hls: add macros for iv and key size

2021-05-11 Thread lance . lmwang
On Tue, May 11, 2021 at 05:04:57PM +0200, Nicolas George wrote: > lance.lmw...@gmail.com (12021-05-11): > > I'm trying to test with AES-192 and AES-256 support. > > Then you will need to differentiate places that are a buffer and you > need to know the maximum size of the key, and places where

Re: [FFmpeg-devel] [PATCH v2] ffmpeg: add -fpsmin to clamp output framerate

2021-05-11 Thread Matthias Neugebauer
Add -fpsmin analogously to -fpsmax for setting a lower bound to the auto-set frame rate. Signed-off-by: Matthias Neugebauer --- doc/ffmpeg.texi | 7 +++ fftools/ffmpeg.c | 8 +++- fftools/ffmpeg.h | 3 +++ fftools/ffmpeg_opt.c | 26 +++--- 4 files changed,

[FFmpeg-devel] [PATCH] ffmpeg: add -fpsmin to clamp output framerate

2021-05-11 Thread Matthias Neugebauer
Add -fpsmin analogously to -fpsmax for setting a lower bound to the auto-set frame rate. --- doc/ffmpeg.texi | 7 +++ fftools/ffmpeg.c | 8 +++- fftools/ffmpeg.h | 3 +++ fftools/ffmpeg_opt.c | 26 +++--- 4 files changed, 40 insertions(+), 4 deletions(-)

Re: [FFmpeg-devel] [PATCH] lavf/flvdec: normalize exporting date metadata

2021-05-11 Thread Anton Khirnov
Quoting Marton Balint (2021-05-10 19:36:59) > > > On Mon, 10 May 2021, Anton Khirnov wrote: > > > Export them in UTC, not the local timezone. This way the output is > > the same everywhere. The timezone information stored in the file is > > still ignored, since there seems to be no simple way

Re: [FFmpeg-devel] [PATCH 1/1] configure: add emscripten support

2021-05-11 Thread Mehdi Sabwat
Hi! Ping, can someone please review? Thanks, Regards On Tue, Apr 27, 2021 at 1:57 AM Mehdi Sabwat wrote: > From: Mehdi Sabwat > > Fix configure test to allow it to have the right suffix when > $target_os=emscripten > --- > configure | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git

Re: [FFmpeg-devel] [PATCH] lavf/flvdec: normalize exporting date metadata

2021-05-11 Thread Anton Khirnov
Quoting Alexander Strasser (2021-05-10 15:35:02) > On 2021-05-10 10:22 +0200, Anton Khirnov wrote: > > Export them in UTC, not the local timezone. This way the output is > > the same everywhere. The timezone information stored in the file is > > still ignored, since there seems to be no simple way

Re: [FFmpeg-devel] [PATCH 08/12] fate/demux: convert flv-demux to ffprobe

2021-05-11 Thread Eoff, Ullysses A
> -Original Message- > From: ffmpeg-devel On Behalf Of Anton > Khirnov > Sent: Sunday, April 25, 2021 12:03 AM > To: ffmpeg-devel@ffmpeg.org > Subject: [FFmpeg-devel] [PATCH 08/12] fate/demux: convert flv-demux to ffprobe > > It can handle side data cleanly. > --- This patch breaks

Re: [FFmpeg-devel] [PATCH] avformat/hls: add macros for iv and key size

2021-05-11 Thread Nicolas George
lance.lmw...@gmail.com (12021-05-11): > I'm trying to test with AES-192 and AES-256 support. Then you will need to differentiate places that are a buffer and you need to know the maximum size of the key, and places where the size you have is the actual key size. Introducing these macros will not

Re: [FFmpeg-devel] [PATCH]: examples/transcode_aac.c - Do not use global pts for frame pts counting

2021-05-11 Thread Nicolas George
James Almer (12021-05-11): > An alternative solution is introducing a struct containing all variables > used by the process, including {input,output}_format_context, > {input,output}_codec_context, resample_context, fifo, and pts, which will > simplify the code a lot by making every function in

Re: [FFmpeg-devel] [PATCH]: examples/transcode_aac.c - Do not use global pts for frame pts counting

2021-05-11 Thread James Almer
On 5/11/2021 10:53 AM, Nicolas George wrote: Ray (12021-05-11): The example transcode_aac.c uses a global pts for counting.  For libavcodec novices this can cause them to overlook this and result with incorrect "start" times of output files if called multiple times (see user error resulting in

Re: [FFmpeg-devel] [PATCH] avformat/hls: add macros for iv and key size

2021-05-11 Thread lance . lmwang
On Tue, May 11, 2021 at 10:56:50AM -0300, James Almer wrote: > On 5/11/2021 10:21 AM, lance.lmw...@gmail.com wrote: > > On Tue, May 11, 2021 at 02:44:34PM +0200, Nicolas George wrote: > > > lance.lmw...@gmail.com (12021-05-11): > > > > From: Limin Wang > > > > > > > > Signed-off-by: Limin Wang

Re: [FFmpeg-devel] [PATCH] avformat/hls: add macros for iv and key size

2021-05-11 Thread James Almer
On 5/11/2021 10:21 AM, lance.lmw...@gmail.com wrote: On Tue, May 11, 2021 at 02:44:34PM +0200, Nicolas George wrote: lance.lmw...@gmail.com (12021-05-11): From: Limin Wang Signed-off-by: Limin Wang --- libavformat/hls.c | 12 ++-- libavformat/hlsenc.c | 19

Re: [FFmpeg-devel] [PATCH] avformat/hls: add macros for iv and key size

2021-05-11 Thread lance . lmwang
On Tue, May 11, 2021 at 02:45:07PM +0200, Nicolas George wrote: > Nicolas George (12021-05-11): > > So, anything using internal.h can only deal with 128 bits keys? Seems > > wrong. > > Are you actually trying to fix something? I'm trying to test with AES-192 and AES-256 support. > > Regards,

Re: [FFmpeg-devel] [PATCH]: examples/transcode_aac.c - Do not use global pts for frame pts counting

2021-05-11 Thread Nicolas George
Ray (12021-05-11): > The example transcode_aac.c uses a global pts for counting.  For > libavcodec novices this can cause them to overlook this and result > with incorrect "start" times of output files if called multiple times > (see user error resulting in bug report >

Re: [FFmpeg-devel] [PATCH] avformat/hls: add macros for iv and key size

2021-05-11 Thread lance . lmwang
On Tue, May 11, 2021 at 02:44:34PM +0200, Nicolas George wrote: > lance.lmw...@gmail.com (12021-05-11): > > From: Limin Wang > > > > Signed-off-by: Limin Wang > > --- > > libavformat/hls.c | 12 ++-- > > libavformat/hlsenc.c | 19 +-- > > libavformat/internal.h |

Re: [FFmpeg-devel] Encode user data unregistered SEI (H.264/H.265)

2021-05-11 Thread Brad Hards
On Saturday, 1 May 2021 12:23:00 PM AEST Brad Hards wrote: > MISB ST 0604 and ST 2101 require user data unregistered SEI messages > (precision timestamps and sensor identifiers) to be included. That > currently isn't supported. This series adds encoding for libx264, > libx265, hevc_nvenc and

[FFmpeg-devel] [PATCH]: examples/transcode_aac.c - Do not use global pts for frame pts counting

2021-05-11 Thread Ray
The example transcode_aac.c uses a global pts for counting.  For libavcodec novices this can cause them to overlook this and result with incorrect "start" times of output files if called multiple times (see user error resulting in bug report https://trac.ffmpeg.org/ticket/9228) From

Re: [FFmpeg-devel] [PATCH] avformat/hls: add macros for iv and key size

2021-05-11 Thread Nicolas George
Nicolas George (12021-05-11): > So, anything using internal.h can only deal with 128 bits keys? Seems > wrong. Are you actually trying to fix something? Regards, -- Nicolas George signature.asc Description: PGP signature ___ ffmpeg-devel mailing

Re: [FFmpeg-devel] [PATCH] avformat/hls: add macros for iv and key size

2021-05-11 Thread Nicolas George
lance.lmw...@gmail.com (12021-05-11): > From: Limin Wang > > Signed-off-by: Limin Wang > --- > libavformat/hls.c | 12 ++-- > libavformat/hlsenc.c | 19 +-- > libavformat/internal.h | 2 ++ > 3 files changed, 17 insertions(+), 16 deletions(-) > > diff --git

[FFmpeg-devel] [PATCH] avformat/hls: add macros for iv and key size

2021-05-11 Thread lance . lmwang
From: Limin Wang Signed-off-by: Limin Wang --- libavformat/hls.c | 12 ++-- libavformat/hlsenc.c | 19 +-- libavformat/internal.h | 2 ++ 3 files changed, 17 insertions(+), 16 deletions(-) diff --git a/libavformat/hls.c b/libavformat/hls.c index

[FFmpeg-devel] [PATCH 1/3] tools/target_dec_fuzzer: Adjust threshold for theora

2021-05-11 Thread Michael Niedermayer
Fixes: Timeout Fixes: 33916/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_THEORA_fuzzer-4620863119949824 Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg Signed-off-by: Michael Niedermayer --- tools/target_dec_fuzzer.c | 2 +- 1 file

[FFmpeg-devel] [PATCH 2/3] avcodec/mjpegdec: Fix order of condition for pal8

2021-05-11 Thread Michael Niedermayer
Fixes: out of array access Fixes: 33960/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_THP_fuzzer-5052852809629696 Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg Signed-off-by: Michael Niedermayer --- libavcodec/mjpegdec.c | 2 +- 1 file

[FFmpeg-devel] [PATCH 3/3] avcodec/jpeglsdec: force pal8 only once

2021-05-11 Thread Michael Niedermayer
Fixes: Infinite loop Fixes: 33958/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_THP_fuzzer-6590264069193728 Fixes: 33981/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_JPEGLS_fuzzer-5343224173559808 Fixes:

[FFmpeg-devel] [PATCH 3/3] lavfi/vf_dnn_processing.c: fix CID 1460603

2021-05-11 Thread Guo, Yejun
CID 1460603 (#1 of 1): Improper use of negative value (NEGATIVE_RETURNS) --- libavfilter/vf_dnn_processing.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/libavfilter/vf_dnn_processing.c b/libavfilter/vf_dnn_processing.c index e05d59a649..e1d9d24683 100644 ---

[FFmpeg-devel] [PATCH 2/3] lavfi/dnn/dnn_io_proc.c: fix Improper use of negative value (NEGATIVE_RETURNS)

2021-05-11 Thread Guo, Yejun
fix coverity CID 1473511 and 1473566 --- libavfilter/dnn/dnn_io_proc.c | 6 ++ 1 file changed, 6 insertions(+) diff --git a/libavfilter/dnn/dnn_io_proc.c b/libavfilter/dnn/dnn_io_proc.c index d5d2654162..02c8e13ed7 100644 --- a/libavfilter/dnn/dnn_io_proc.c +++

[FFmpeg-devel] [PATCH 1/3] lavfi/dnn/dnn_io_proc.c: Fix Out-of-bounds access (ARRAY_VS_SINGLETON)

2021-05-11 Thread Guo, Yejun
fix coverity CID 1473571, 1473577 and 1482089 --- libavfilter/dnn/dnn_io_proc.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/libavfilter/dnn/dnn_io_proc.c b/libavfilter/dnn/dnn_io_proc.c index 1e2bef3f9a..d5d2654162 100644 --- a/libavfilter/dnn/dnn_io_proc.c +++

Re: [FFmpeg-devel] [PATCH v3 2/2] avformat: add data_size for ff_hex_to_data()

2021-05-11 Thread lance . lmwang
On Tue, May 11, 2021 at 05:15:46AM +0200, Andreas Rheinhardt wrote: > lance.lmw...@gmail.com: > > On Tue, May 11, 2021 at 03:45:51AM +0200, Andreas Rheinhardt wrote: > >> lance.lmw...@gmail.com: > >>> On Tue, May 11, 2021 at 01:27:09AM +0200, Andreas Rheinhardt wrote: >

Re: [FFmpeg-devel] [PATCH v3 2/2] avformat: add data_size for ff_hex_to_data()

2021-05-11 Thread Nicolas George
lance.lmw...@gmail.com (12021-05-11): > Maybe I think it's better to alloc in ff_hex_to_data function and return the > buffer directly. Then certainly you are wrong. You should never suggest to use a dynamic allocation when the code can be written with automatic allocation. Please remember that