Re: [FFmpeg-devel] [PATCH v3 2/2] cmdutils: add show_help_protocol for get protocol options

2019-12-28 Thread Liu Steven
> 在 2019年11月28日,上午5:24,Michael Niedermayer 写道: > > On Wed, Nov 27, 2019 at 01:57:02PM +0800, Steven Liu wrote: >> Signed-off-by: Steven Liu >> --- >> fftools/cmdutils.c | 14 ++ >> fftools/ffmpeg_opt.c | 2 +- >> 2 files changed, 15 insertions(+), 1 deletion(-) > > i think the

Re: [FFmpeg-devel] [PATCH 2/2] lavc/vp9: support hardware decode with resolution changing on inter frame

2019-12-28 Thread Fu, Linjie
> -Original Message- > From: ffmpeg-devel On Behalf Of > Hendrik Leppkes > Sent: Friday, December 27, 2019 18:22 > To: FFmpeg development discussions and patches de...@ffmpeg.org> > Subject: Re: [FFmpeg-devel] [PATCH 2/2] lavc/vp9: support hardware > decode with resolution changing on

Re: [FFmpeg-devel] [PATCH v2 3/4] FATE: add a test for lut1d and lut3d

2019-12-28 Thread Limin Wang
On Thu, Nov 14, 2019 at 10:55:30PM +0100, Michael Niedermayer wrote: > On Thu, Nov 14, 2019 at 09:46:22PM +0800, lance.lmw...@gmail.com wrote: > > From: Limin Wang > > > > Signed-off-by: Limin Wang > > --- > > have tested with x86_64(apple darwin, linux gcc), x86_32(linux), mips > > tested on

[FFmpeg-devel] [PATCH V2] lavf/libsrt: add version guard for srt encryption control

2019-12-28 Thread Jun Zhao
From: Jun Zhao add version guard for srt encryption control. and use SRTO_STRICTENC(53) for compatibility. Signed-off-by: Jun Zhao --- libavformat/libsrt.c | 11 +-- 1 files changed, 9 insertions(+), 2 deletions(-) diff --git a/libavformat/libsrt.c b/libavformat/libsrt.c index

Re: [FFmpeg-devel] [PATCH 2/2] avformat/matroskadec: Fix lzo decompression

2019-12-28 Thread James Almer
On 12/27/2019 11:40 PM, Andreas Rheinhardt wrote: > When a Matroska Block is only stored in compressed form, the size of > the uncompressed block is not explicitly coded and therefore not known > before decompressing it. Therefore the demuxer uses a guess for the > uncompressed size: The first

Re: [FFmpeg-devel] [PATCH 1/2] fate/matroska: Add test for ProRes and bz2 compression

2019-12-28 Thread James Almer
On 12/27/2019 11:40 PM, Andreas Rheinhardt wrote: > This test tests that demuxing ProRes that is muxed like it should be in > Matroska (i.e. with the first header ("icpf") atom stripped away) works; > it also tests bz2 decompression as well as the handling of > unknown-length clusters. > >

[FFmpeg-devel] [PATCH 2/3] avcodec/libx265: add a qp option and apply the relevant global AVCodecContext settings to the encoder context

2019-12-28 Thread James Almer
Signed-off-by: James Almer --- doc/encoders.texi| 22 ++ libavcodec/libx265.c | 28 2 files changed, 50 insertions(+) diff --git a/doc/encoders.texi b/doc/encoders.texi index 673eaf6496..61e674cf96 100644 --- a/doc/encoders.texi +++

[FFmpeg-devel] [PATCH 3/3] avcodec/libx265: export encoded frame stats

2019-12-28 Thread James Almer
Signed-off-by: James Almer --- libavcodec/libx265.c | 16 +++- 1 file changed, 11 insertions(+), 5 deletions(-) diff --git a/libavcodec/libx265.c b/libavcodec/libx265.c index 581732d169..68eba17fca 100644 --- a/libavcodec/libx265.c +++ b/libavcodec/libx265.c @@ -416,6 +416,7 @@

[FFmpeg-devel] [PATCH 1/3] avcodec/libx265: apply some global AVCodecContext settings to the encoder context

2019-12-28 Thread James Almer
There's no reason to ignore them if set. Signed-off-by: James Almer --- doc/encoders.texi| 14 ++ libavcodec/libx265.c | 13 + 2 files changed, 27 insertions(+) diff --git a/doc/encoders.texi b/doc/encoders.texi index a207363650..673eaf6496 100644 ---

[FFmpeg-devel] [PATCH 2/2] tools/target_dec_fuzzer: List valid codec tags (based on fate)

2019-12-28 Thread Michael Niedermayer
I am a bit undecided on listing them like this but it seems the fuzzer has difficulty finding valid tags (like in hapdec/snappy) With this it finds issues in hapdec within seconds locally (with constrained w/h) while before on googles machienes it seemed not to get past the codec_tag switch at

[FFmpeg-devel] [PATCH 1/2] tools/target_dec_fuzzer: Stop negative block_align and sampling rate

2019-12-28 Thread Michael Niedermayer
These are checked for early in avcodec_open2() and do not really test the decoder but instead waste resources which could be better spend fuzzing the actual decoder Signed-off-by: Michael Niedermayer --- tools/target_dec_fuzzer.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff

[FFmpeg-devel] [PATCH] lavc/qsv: remove the unused funtion ff_qsv_profile_to_mfx()

2019-12-28 Thread Zhong Li
Signed-off-by: Zhong Li --- libavcodec/qsv.c | 52 --- libavcodec/qsv_internal.h | 1 - 2 files changed, 53 deletions(-) diff --git a/libavcodec/qsv.c b/libavcodec/qsv.c index b00e427435..db98c75073 100644 --- a/libavcodec/qsv.c +++

Re: [FFmpeg-devel] [PATCH] avformat/ftp: Exit with error on ftp_open if file does not exist

2019-12-28 Thread Fotyev V.
See attached On 2019-12-28 17:17, Michael Niedermayer wrote: On Sat, Dec 28, 2019 at 01:52:04PM +0300, Fotyev V. wrote: Add a check for code 550 when requesting file size ---  libavformat/ftp.c | 11 +--  1 file changed, 9 insertions(+), 2 deletions(-) diff --git a/libavformat/ftp.c

[FFmpeg-devel] [PATCH 1/3] avutil/eval: separate AVExpr state to a new AVExprState struct

2019-12-28 Thread Marton Balint
Also add helper functions to allocate and free such a struct, and make it usable by providing a new av_eval_expr2 function for which you can specify a custom AVExprState. Signed-off-by: Marton Balint --- doc/APIchanges | 3 +++ libavutil/eval.c| 31 +--

[FFmpeg-devel] [PATCH 3/3] avfilter/vf_geq: fix multiple assignments of ptr in slice_geq_filter

2019-12-28 Thread Marton Balint
Fixes Coverity CID 1427183. Signed-off-by: Marton Balint --- libavfilter/vf_geq.c | 10 -- 1 file changed, 4 insertions(+), 6 deletions(-) diff --git a/libavfilter/vf_geq.c b/libavfilter/vf_geq.c index 417b9d..00f02b6ebf 100644 --- a/libavfilter/vf_geq.c +++ b/libavfilter/vf_geq.c

[FFmpeg-devel] [PATCH 2/3] avfilter/vf_geq: use per-thread state for expression evaluation

2019-12-28 Thread Marton Balint
Fixes ticket #7528. Signed-off-by: Marton Balint --- libavfilter/vf_geq.c | 9 +++-- 1 file changed, 7 insertions(+), 2 deletions(-) diff --git a/libavfilter/vf_geq.c b/libavfilter/vf_geq.c index 2905efae24..417b9d 100644 --- a/libavfilter/vf_geq.c +++ b/libavfilter/vf_geq.c @@ -377,6

Re: [FFmpeg-devel] [PATCH v4] avfilter/buffersrc: deprecate sws_param option

2019-12-28 Thread Nicolas George
zhilizhao (12019-12-28): > Could you help apply the patch? Since mypopy does not explain themselves, I will apply. But the patch needs to be rebased and conflicts resolved. Regards, -- Nicolas George signature.asc Description: PGP signature ___

[FFmpeg-devel] [PATCH 2/2] lavc/qsvdec: disable gpu copy when not supported

2019-12-28 Thread Zhong Li
Signed-off-by: Zhong Li --- libavcodec/qsvdec.c | 6 -- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/libavcodec/qsvdec.c b/libavcodec/qsvdec.c index baaf1f205f..fc25dc73e5 100644 --- a/libavcodec/qsvdec.c +++ b/libavcodec/qsvdec.c @@ -99,9 +99,11 @@ static int

[FFmpeg-devel] [PATCH 1/2] lavc/qsvdec: fix the error retrun code in ff_qsv_get_continuous_buffer()

2019-12-28 Thread Zhong Li
Signed-off-by: Zhong Li --- libavcodec/qsvdec.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavcodec/qsvdec.c b/libavcodec/qsvdec.c index 0d34021b42..baaf1f205f 100644 --- a/libavcodec/qsvdec.c +++ b/libavcodec/qsvdec.c @@ -74,7 +74,7 @@ static int

Re: [FFmpeg-devel] [PATCH] avformat/ftp: Exit with error on ftp_open if file does not exist

2019-12-28 Thread Michael Niedermayer
On Sat, Dec 28, 2019 at 01:52:04PM +0300, Fotyev V. wrote: > Add a check for code 550 when requesting file size > --- >  libavformat/ftp.c | 11 +-- >  1 file changed, 9 insertions(+), 2 deletions(-) > > diff --git a/libavformat/ftp.c b/libavformat/ftp.c > index 97ad80d..64a5250 100644 >

Re: [FFmpeg-devel] [PATCH 1/5] avcodec/atrac9dec: Clamp band_ext_data to max that can be read if skipped.

2019-12-28 Thread Michael Niedermayer
On Sat, Dec 28, 2019 at 02:03:37PM +0100, Lynne wrote: > Dec 27, 2019, 21:35 by mich...@niedermayer.cc: > > > On Thu, Dec 26, 2019 at 05:24:31PM +0100, Lynne wrote: > > > >> Dec 26, 2019, 13:57 by d...@lynne.ee: > >> > >> > Dec 16, 2019, 23:19 by mich...@niedermayer.cc: > >> > > >> >> Fixes: out

Re: [FFmpeg-devel] [PATCH v4] avfilter/buffersrc: deprecate sws_param option

2019-12-28 Thread zhilizhao
Hi Nicolas, > On Dec 23, 2019, at 8:23 PM, Nicolas George wrote: > > myp...@gmail.com (12019-12-12): >> LGTM, but need to wait next release cycle? > > What do you mean? There is all the necessary code to let the option > disappear automatically at the next release. > > I shall push soon,

Re: [FFmpeg-devel] [PATCH 1/5] avcodec/atrac9dec: Clamp band_ext_data to max that can be read if skipped.

2019-12-28 Thread Lynne
Dec 27, 2019, 21:35 by mich...@niedermayer.cc: > On Thu, Dec 26, 2019 at 05:24:31PM +0100, Lynne wrote: > >> Dec 26, 2019, 13:57 by d...@lynne.ee: >> >> > Dec 16, 2019, 23:19 by mich...@niedermayer.cc: >> > >> >> Fixes: out of array read >> >> Fixes: >> >>

[FFmpeg-devel] [PATCH] avformat/ftp: Exit with error on ftp_open if file does not exist

2019-12-28 Thread Fotyev V.
Add a check for code 550 when requesting file size ---  libavformat/ftp.c | 11 +--  1 file changed, 9 insertions(+), 2 deletions(-) diff --git a/libavformat/ftp.c b/libavformat/ftp.c index 97ad80d..64a5250 100644 --- a/libavformat/ftp.c +++ b/libavformat/ftp.c @@ -391,13 +391,17 @@

[FFmpeg-devel] [PATCHv2 8/8] avformat/img2enc: add support for specifying protocol options

2019-12-28 Thread Marton Balint
v2: simplified example Signed-off-by: Marton Balint --- doc/muxers.texi | 11 +++ libavformat/img2enc.c | 13 - 2 files changed, 23 insertions(+), 1 deletion(-) diff --git a/doc/muxers.texi b/doc/muxers.texi index fb5c9bc4c0..470a11af95 100644 --- a/doc/muxers.texi

Re: [FFmpeg-devel] [PATCH] lavf/libsrt: add version guard for srt encryption control

2019-12-28 Thread Marton Balint
On Sat, 28 Dec 2019, Jun Zhao wrote: From: Jun Zhao add version guard for srt encryption control. Ok, but the code will still fail in 1.3.2 because that version has no constant named SRTO_ENFORCEDENCRYPTION. So I suggest you use the original name for it, SRTO_STRICTENC. Regards,

[FFmpeg-devel] [PATCH] lavf/libsrt: add version guard for srt encryption control

2019-12-28 Thread Jun Zhao
From: Jun Zhao add version guard for srt encryption control. Signed-off-by: Jun Zhao --- libavformat/libsrt.c |6 ++ 1 files changed, 6 insertions(+), 0 deletions(-) diff --git a/libavformat/libsrt.c b/libavformat/libsrt.c index 2fdfe8e..f981ef1 100644 --- a/libavformat/libsrt.c +++

[FFmpeg-devel] [PATCH] doc/filters: Document geq *sum functions

2019-12-28 Thread Michael Niedermayer
Signed-off-by: Michael Niedermayer --- doc/filters.texi | 4 1 file changed, 4 insertions(+) diff --git a/doc/filters.texi b/doc/filters.texi index 57330d1fd9..4d255d57c6 100644 --- a/doc/filters.texi +++ b/doc/filters.texi @@ -11374,6 +11374,10 @@ red/green/blue component. Return 0 if