[FFmpeg-devel] [PATCH] avfilter/fps: remove unconventional acronyms

2021-07-02 Thread Gyan Doshi
In dd770883e9, support for expressions was added. Among the constants added were labels of qnstc, qpal, sntsc & spal. These were added in ba2a8cb40b to represent parameter permutations where only the resolution is different. They don't have any usage currency and don't represent any industry stand

Re: [FFmpeg-devel] Speech Recognition Filter Idea

2021-07-02 Thread Harry Muscle
Would there be a better mailing list to get feedback on this idea and find out if it's viable for inclusion? Thanks, Harry On Thu, Jul 1, 2021, 11:45 PM Harry Muscle, wrote: > I'm thinking of developing a speech recognition audio filter for ffmpeg. > The idea would be that this filter would sea

Re: [FFmpeg-devel] [PATCH 1/3] avcodec/lpc: check for zero err in normalization in compute_lpc_coefs()

2021-07-02 Thread Michael Niedermayer
On Sat, Jun 05, 2021 at 06:12:47PM +0200, Anton Khirnov wrote: > Quoting Michael Niedermayer (2021-06-05 16:14:24) > > On Sat, Jun 05, 2021 at 03:44:49PM +0200, Anton Khirnov wrote: > > > Quoting Michael Niedermayer (2021-06-01 09:33:13) > > > > Fixes: floating point division by 0 > > > > Fixes: Ti

Re: [FFmpeg-devel] [PATCH 3/3] avcodec/j2kenc: Check for av_strtok() failure

2021-07-02 Thread Michael Niedermayer
On Tue, May 18, 2021 at 11:39:10PM +0200, Michael Niedermayer wrote: > Fixes: CID1466601 Dereference null return value > > Signed-off-by: Michael Niedermayer > --- > libavcodec/j2kenc.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) will apply [...] -- Michael GnuPG fingerprint:

Re: [FFmpeg-devel] [PATCH 2/3] avformat/ftp: Check for av_strtok() failure

2021-07-02 Thread Michael Niedermayer
On Tue, May 18, 2021 at 11:39:09PM +0200, Michael Niedermayer wrote: > Fixes: CID1396258 Dereference null return value > > Signed-off-by: Michael Niedermayer > --- > libavformat/ftp.c | 2 ++ > 1 file changed, 2 insertions(+) will apply [...] -- Michael GnuPG fingerprint: 9FF2128B147EF673

Re: [FFmpeg-devel] [PATCH 2/3] tools/cws2fws: Check read() for failure

2021-07-02 Thread Michael Niedermayer
On Sun, May 16, 2021 at 09:39:00PM +0200, Michael Niedermayer wrote: > Fixes: CID1452579 Argument cannot be negative > > Signed-off-by: Michael Niedermayer > --- > tools/cws2fws.c | 6 ++ > 1 file changed, 6 insertions(+) will apply [...] -- Michael GnuPG fingerprint: 9FF2128B147EF673

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

2021-07-02 Thread Michael Niedermayer
On Tue, May 11, 2021 at 01:12:04PM +0200, Michael Niedermayer wrote: > 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 > Sign

Re: [FFmpeg-devel] [PATCH 1/3] avformat/flv: add support for h265 streams

2021-07-02 Thread Zhao Zhili
> On Jul 2, 2021, at 3:21 AM, Martin Storsjö wrote: > > On Thu, 1 Jul 2021, Jan Ekström wrote: > >> On Thu, Jul 1, 2021 at 5:16 PM James Almer wrote: >>> >>> From: Matthieu Patou >>> >>> Suggested-by: ffm...@fb.com >>> Signed-off-by: James Almer >>> --- >> >> For the record, if Big Compa

Re: [FFmpeg-devel] [PATCH 1/3] avformat/matroskadec: Reset state also on failure in matroska_reset_status()

2021-07-02 Thread Andreas Rheinhardt
Michael Niedermayer: > The calling code does not handle failures and will fail with assertion > failures later. > Seeking can always fail even when the position was previously read. > > Fixes: Assertion failure > Fixes: > 35253/clusterfuzz-testcase-minimized-ffmpeg_dem_MATROSKA_fuzzer-4693059982

Re: [FFmpeg-devel] [PATCH v2 3/3] avcodec/libx264: remove separate libx264rgb RGB wrapper

2021-07-02 Thread Jan Ekström
On Fri, Jul 2, 2021 at 6:25 PM Michael Niedermayer wrote: > > On Fri, Jul 02, 2021 at 02:25:13PM +0300, Jan Ekström wrote: > > No other encoder wrapper in libavcodec is split like this, and > > with RGB input this currently does not lead to 4:2:0 (which would > > be generally supported in most har

Re: [FFmpeg-devel] [PATCH v2 3/3] avcodec/libx264: remove separate libx264rgb RGB wrapper

2021-07-02 Thread Michael Niedermayer
On Fri, Jul 02, 2021 at 02:25:13PM +0300, Jan Ekström wrote: > No other encoder wrapper in libavcodec is split like this, and > with RGB input this currently does not lead to 4:2:0 (which would > be generally supported in most hardware and software implementations), > but rather 4:4:4. > > The lib

Re: [FFmpeg-devel] [PATCH v2 2/3] {configure, avcodec/libx264}: remove separate x264_csp_bgr check

2021-07-02 Thread Jan Ekström
On Fri, Jul 2, 2021 at 3:10 PM Jan Ekström wrote: > > On Fri, Jul 2, 2021 at 2:25 PM Jan Ekström wrote: > > > > We already require X264_BUILD >= 118, which includes an unconditional > > definition of X264_CSP_BGR in itself, thus making this check > > effectively always true. > > --- > > configur

Re: [FFmpeg-devel] [PATCH v2 2/3] {configure, avcodec/libx264}: remove separate x264_csp_bgr check

2021-07-02 Thread Jan Ekström
On Fri, Jul 2, 2021 at 2:25 PM Jan Ekström wrote: > > We already require X264_BUILD >= 118, which includes an unconditional > definition of X264_CSP_BGR in itself, thus making this check > effectively always true. > --- > configure| 3 +-- > libavcodec/libx264.c | 7 +-- > 2 files

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

2021-07-02 Thread James Almer
On 7/2/2021 2:55 AM, Wang, Fei W wrote: On Thu, 2021-07-01 at 09:41 -0300, James Almer wrote: On 6/17/2021 3:10 AM, Fei Wang wrote: shifted_order_hints is computed by data with int plus data with int. Switch to int8_t may lose its precision. Signed-off-by: Fei Wang --- libavcodec/cbs_av1_s

[FFmpeg-devel] [PATCH v2 2/3] {configure, avcodec/libx264}: remove separate x264_csp_bgr check

2021-07-02 Thread Jan Ekström
We already require X264_BUILD >= 118, which includes an unconditional definition of X264_CSP_BGR in itself, thus making this check effectively always true. --- configure| 3 +-- libavcodec/libx264.c | 7 +-- 2 files changed, 2 insertions(+), 8 deletions(-) diff --git a/configure b

[FFmpeg-devel] [PATCH v2 1/3] configure: move x264_csp_bgr check under general libx264 checks

2021-07-02 Thread Jan Ekström
This makes the libx264rgb check work when pkg-config is utilized and x264.h is not part of the standard include path (as is often with cross-compilation, or when you just have a custom prefix in general in f.ex. your home directory). The X264_BUILD >= 118 required by configure since 2011 should ha

[FFmpeg-devel] [PATCH v2 3/3] avcodec/libx264: remove separate libx264rgb RGB wrapper

2021-07-02 Thread Jan Ekström
No other encoder wrapper in libavcodec is split like this, and with RGB input this currently does not lead to 4:2:0 (which would be generally supported in most hardware and software implementations), but rather 4:4:4. The libx262 encoder definition was not touched, as it already has 4:4:4 YCbCr de

[FFmpeg-devel] [PATCH v2 0/3] libx264 configure check clean-up, removal of libx264rgb

2021-07-02 Thread Jan Ekström
Changes compared to v1: - As the ff_libx264rgb_encoder extern usage depended on the configure define being set to a falsie, move the actual removal of the configure entry for it to the change which removes libx264rgb altogether. Overall resulting diff between v1 and v2 is zero. Done in three

[FFmpeg-devel] [PATCH] ffmpeg: add option recast_media

2021-07-02 Thread Gyan Doshi
Allows forcing decoders of different media type. Needed to decode media data muxed as data streams. --- doc/ffmpeg.texi | 5 + fftools/ffmpeg_opt.c | 7 ++- 2 files changed, 11 insertions(+), 1 deletion(-) diff --git a/doc/ffmpeg.texi b/doc/ffmpeg.texi index 7827291755..c1065086e5 10

Re: [FFmpeg-devel] [PATCH 3/9] avcodec/av1dec: support setup shear process

2021-07-02 Thread Wang, Fei W
On Thu, 2021-07-01 at 09:39 -0300, James Almer wrote: > On 6/17/2021 3:10 AM, Fei Wang wrote: > > Defined in spec 7.11.3.6/7.11.3.7. > > > > Signed-off-by: Fei Wang > > --- > > libavcodec/av1dec.c | 97 > > + > > libavcodec/av1dec.h | 1 + > > 2 fi