Re: [FFmpeg-devel] [PATCH 1/5] avfilter/af_afade: fix crossfade duration maximum value

2018-09-30 Thread Gyan
On Mon, Oct 1, 2018 at 2:15 AM Marton Balint wrote: > > +{ "duration", "set cross fade duration", > OFFSET(duration), AV_OPT_TYPE_DURATION, {.i64 = 0. }, 0, 6000, > FLAGS }, > +{ "d","set cross fade duration", > OFFSET(duration), AV_OPT_TYPE_DURATION, {.i64 =

[FFmpeg-devel] [PATCH v2] avcodec: libdav1d AV1 decoder wrapper.

2018-09-30 Thread James Almer
From: Ronald S. Bultje Originally written by Ronald S. Bultje, with fixes, optimizations and improvements by James Almer. Signed-off-by: James Almer --- Updated with some refactoring and to use a few new public facing fields recently introduced. The API is unstable and in constant development,

[FFmpeg-devel] [PATCH v2 4/5] h265_metadata: Add option to set the level of the stream

2018-09-30 Thread Mark Thompson
--- doc/bitstream_filters.texi | 9 +++ libavcodec/h265_metadata_bsf.c | 136 + 2 files changed, 145 insertions(+) diff --git a/doc/bitstream_filters.texi b/doc/bitstream_filters.texi index d948c6d658..98b7102309 100644 --- a/doc/bitstream_filters.texi +++ b/

[FFmpeg-devel] [PATCH v2 2/5] lavc/h265_profile_level: Fix DPB size calculation

2018-09-30 Thread Mark Thompson
The maxDpbPicBuf value which is used in the DPB size calculation depends on the profile (it's usually 6, but 7 for screen-extended profiles). --- libavcodec/h265_profile_level.c | 86 - libavcodec/h265_profile_level.h | 1 + 2 files changed, 44 insertions(+), 43 de

[FFmpeg-devel] [PATCH v2 5/5] lavc/h265_profile_level: Add unit test

2018-09-30 Thread Mark Thompson
Operates in the same way as the h264-levels test. --- libavcodec/Makefile| 1 + libavcodec/tests/h265_levels.c | 297 + tests/fate/libavcodec.mak | 5 + 3 files changed, 303 insertions(+) create mode 100644 libavcodec/tests/h265_levels.c diff

[FFmpeg-devel] [PATCH v2 3/5] lavc/h265_profile_level: Do not allow high tier at level < 4

2018-09-30 Thread Mark Thompson
--- libavcodec/h265_profile_level.c | 18 -- 1 file changed, 12 insertions(+), 6 deletions(-) diff --git a/libavcodec/h265_profile_level.c b/libavcodec/h265_profile_level.c index 692252bb4d..126f8dbc2a 100644 --- a/libavcodec/h265_profile_level.c +++ b/libavcodec/h265_profile_leve

[FFmpeg-devel] [PATCH v2 1/5] fate/libavcodec: Fix config dependency of h264-levels test

2018-09-30 Thread Mark Thompson
--- tests/fate/libavcodec.mak | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tests/fate/libavcodec.mak b/tests/fate/libavcodec.mak index aa4c36b112..5dde1243fa 100644 --- a/tests/fate/libavcodec.mak +++ b/tests/fate/libavcodec.mak @@ -46,7 +46,7 @@ fate-dct8x8: libavcodec/test

Re: [FFmpeg-devel] [PATCH 4/4] lavc/h265_profile_level: Add unit test

2018-09-30 Thread Mark Thompson
On 27/09/18 20:58, Michael Niedermayer wrote: > On Thu, Sep 27, 2018 at 12:17:07AM +0100, Mark Thompson wrote: >> Operates in the same way as the h264-levels test. >> --- >> libavcodec/Makefile| 1 + >> libavcodec/tests/h265_levels.c | 297 + >> tests/

[FFmpeg-devel] [PATCH 2/2] avfilter/f_cue: use inlink fifo for queueing frames

2018-09-30 Thread Marton Balint
Signed-off-by: Marton Balint --- libavfilter/f_cue.c | 87 ++--- 1 file changed, 29 insertions(+), 58 deletions(-) diff --git a/libavfilter/f_cue.c b/libavfilter/f_cue.c index 732b5e218a..9cf710c6d2 100644 --- a/libavfilter/f_cue.c +++ b/libavfilte

[FFmpeg-devel] [PATCH 1/2] avfilter/filters: add ff_inlink_peek_frame and ff_inlink_queued_frames to access frames in the inlink fifo

2018-09-30 Thread Marton Balint
Signed-off-by: Marton Balint --- libavfilter/avfilter.c | 13 + libavfilter/filters.h | 14 ++ 2 files changed, 27 insertions(+) diff --git a/libavfilter/avfilter.c b/libavfilter/avfilter.c index 85eff0aa1d..9e4b8e5ca3 100644 --- a/libavfilter/avfilter.c +++ b/libavfilte

Re: [FFmpeg-devel] [PATCH] avformat: add H264 and HEVC support in IVF muxer

2018-09-30 Thread Jan Ekström
On Mon, Oct 1, 2018 at 12:01 AM Jan Ekström wrote: > > Additionally, please make sure that the files created with these > changes can be demuxed with the FFmpeg IVF demuxer (ivfdec.c). I would > guess the H.264 stuff would work, but since the IVF demuxer utilizes > the ff_codec_bmp_tags list, whic

Re: [FFmpeg-devel] [PATCH] avformat: add H264 and HEVC support in IVF muxer

2018-09-30 Thread James Almer
On 9/30/2018 6:01 PM, Jan Ekström wrote: > Sn Sun, Sep 30, 2018 at 8:56 PM wrote: >> >> ... >> +if (st->codecpar->codec_id == AV_CODEC_ID_H264) { >> +if (pkt->size >= 5 && AV_RB32(pkt->data) != 0x001 && >> + (AV_RB24(pkt->data) != 0x01 || >> +

Re: [FFmpeg-devel] [PATCH] avformat: add H264 and HEVC support in IVF muxer

2018-09-30 Thread Jan Ekström
Sn Sun, Sep 30, 2018 at 8:56 PM wrote: > > ... > +if (st->codecpar->codec_id == AV_CODEC_ID_H264) { > +if (pkt->size >= 5 && AV_RB32(pkt->data) != 0x001 && > + (AV_RB24(pkt->data) != 0x01 || > + (st->codecpar->extradata_

Re: [FFmpeg-devel] [PATCH] avformat: add H264 and HEVC support in IVF muxer

2018-09-30 Thread Nicolas George
Paul B Mahol (2018-09-30): > > For consistency within a single function, an assert is the correct > > choice. > No, it is not. Asserts do not add any consistency. Asserts do not add consistency indeed. Their purpose is to check consistency, which is exactly what needs testing here: the consistency

[FFmpeg-devel] [PATCH 5/5] avutil/parseutils: fix some overflows in duration calculations

2018-09-30 Thread Marton Balint
Also properly return AVERROR(ERANGE) in case of actual overflows. Signed-off-by: Marton Balint --- libavutil/parseutils.c| 14 ++ tests/ref/fate/parseutils | 8 2 files changed, 14 insertions(+), 8 deletions(-) diff --git a/libavutil/parseutils.c b/libavutil/parseutils

[FFmpeg-devel] [PATCH 4/5] avutil/tests/parseutils: add some big duration tests

2018-09-30 Thread Marton Balint
These are buggy for now... Signed-off-by: Marton Balint --- libavutil/tests/parseutils.c | 4 tests/ref/fate/parseutils| 4 2 files changed, 8 insertions(+) diff --git a/libavutil/tests/parseutils.c b/libavutil/tests/parseutils.c index 180f624002..a1ac8d44e2 100644 --- a/libavutil

[FFmpeg-devel] [PATCH 2/5] avformat/segment: fix segment_time_delta option min/max value

2018-09-30 Thread Marton Balint
Signed-off-by: Marton Balint --- libavformat/segment.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavformat/segment.c b/libavformat/segment.c index 7fb4dc7d21..e2ac2c3a7f 100644 --- a/libavformat/segment.c +++ b/libavformat/segment.c @@ -1045,7 +1045,7 @@ static const A

[FFmpeg-devel] [PATCH 3/5] avutil/opt: check for minimum and maximum values when setting AV_OPT_TYPE_DURATION fields

2018-09-30 Thread Marton Balint
Signed-off-by: Marton Balint --- libavutil/opt.c | 21 ++--- 1 file changed, 14 insertions(+), 7 deletions(-) diff --git a/libavutil/opt.c b/libavutil/opt.c index 99282605f5..93d6c26c11 100644 --- a/libavutil/opt.c +++ b/libavutil/opt.c @@ -496,15 +496,22 @@ int av_opt_set(void *

[FFmpeg-devel] [PATCH 1/5] avfilter/af_afade: fix crossfade duration maximum value

2018-09-30 Thread Marton Balint
Signed-off-by: Marton Balint --- libavfilter/af_afade.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/libavfilter/af_afade.c b/libavfilter/af_afade.c index d823e82d39..c517b4117c 100644 --- a/libavfilter/af_afade.c +++ b/libavfilter/af_afade.c @@ -354,8 +354,8 @@ AVFilte

Re: [FFmpeg-devel] [PATCH] avformat: add H264 and HEVC support in IVF muxer

2018-09-30 Thread Paul B Mahol
On 9/30/18, Nicolas George wrote: > Paul B Mahol (2018-09-30): >> No, AVERROR_BUG is better. > > For consistency within a single function, an assert is the correct > choice. No, it is not. Asserts do not add any consistency. ___ ffmpeg-devel mailing lis

Re: [FFmpeg-devel] Mentoring project: music test source

2018-09-30 Thread Mina
On 09/30/2018 07:41 PM, Nicolas George wrote: Hi. For the next rounds of sponsored internships, I would like to propose the following project, that I would mentor: A music-like audio lavfi source for testing purposes. That means a deterministic pseudo-random stream of notes with varied frequ

Re: [FFmpeg-devel] [PATCH] ffmpeg: break loop when dts_heuristic check done

2018-09-30 Thread Michael Niedermayer
On Sun, Sep 30, 2018 at 08:20:07PM +0800, xiaofeng wrote: > Signed-off-by: xiaofeng > > -- > ffmpeg_opt.c |4 +++- > 1 file changed, 3 insertions(+), 1 deletion(-) > 1b5052ace797da32bf90a23aa771f020159fa4d2 > 0001-ffmpeg-break-loop-when-dts_heuristic-check-done.patch > From 8fd783a0a4e780

Re: [FFmpeg-devel] [PATCH 1/2] avcodec/wcmv: Fix integer overflows

2018-09-30 Thread Michael Niedermayer
On Tue, Sep 25, 2018 at 03:56:40AM +0200, Michael Niedermayer wrote: > Fixes: signed integer overflow: 262140 * 65535 cannot be represented in type > 'int' > Fixes: > 10090/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_WCMV_fuzzer-5691269368512512 > > Found-by: continuous fuzzing process >

[FFmpeg-devel] [PATCH] lavfi/vf_colorconstancy: add weighted_greyedge

2018-09-30 Thread Mina
Hi,   This patch was part of GSoC Color Constancy filter. It introduces an improved color constancy filter(weighted_greyedge) based on the already pushed greyedge. I'm sending it again after updating it against latest version of master. Regards, Mina Sami >From 33b92a766ff575c9765c15097cdda

Re: [FFmpeg-devel] [PATCH] avformat: add H264 and HEVC support in IVF muxer

2018-09-30 Thread Nicolas George
Paul B Mahol (2018-09-30): > No, AVERROR_BUG is better. For consistency within a single function, an assert is the correct choice. Regards, -- Nicolas George signature.asc Description: Digital signature ___ ffmpeg-devel mailing list ffmpeg-devel@f

Re: [FFmpeg-devel] [PATCH] avformat: add H264 and HEVC support in IVF muxer

2018-09-30 Thread James Almer
On 9/28/2018 3:51 PM, alx.sukha...@gmail.com wrote: > From: Alex Sukhanov > > --- > libavformat/ivfenc.c | 41 - > 1 file changed, 36 insertions(+), 5 deletions(-) > > diff --git a/libavformat/ivfenc.c b/libavformat/ivfenc.c > index 66441a2a43..68c36daf09

Re: [FFmpeg-devel] [PATCH] avformat: add H264 and HEVC support in IVF muxer

2018-09-30 Thread Paul B Mahol
On 9/30/18, Carl Eugen Hoyos wrote: > 2018-09-28 20:51 GMT+02:00, alx.sukha...@gmail.com : >> From: Alex Sukhanov > >> if (par->codec_type != AVMEDIA_TYPE_VIDEO || >> !(par->codec_id == AV_CODEC_ID_AV1 || >> + par->codec_id == AV_CODEC_ID_H264 || >> + par->codec_id

Re: [FFmpeg-devel] [PATCH] avformat: add H264 and HEVC support in IVF muxer

2018-09-30 Thread Carl Eugen Hoyos
2018-09-28 20:51 GMT+02:00, alx.sukha...@gmail.com : > From: Alex Sukhanov > if (par->codec_type != AVMEDIA_TYPE_VIDEO || > !(par->codec_id == AV_CODEC_ID_AV1 || > + par->codec_id == AV_CODEC_ID_H264 || > + par->codec_id == AV_CODEC_ID_HEVC || >par->cod

Re: [FFmpeg-devel] [PATCH] avdevice/decklink: Add option to align Capture start time

2018-09-30 Thread Marton Balint
On Fri, 28 Sep 2018, Jeyapal, Karthick wrote: On 9/28/18 2:58 AM, Marton Balint wrote: On Mon, 24 Sep 2018, Karthick J wrote: From: Karthick Jeyapal This option is useful for maintaining input synchronization across N different hardware devices deployed for 'N-way' redundancy. The syst

Re: [FFmpeg-devel] Mentoring project: music test source

2018-09-30 Thread Carl Eugen Hoyos
2018-09-30 19:41 GMT+02:00, Nicolas George : > For the next rounds of sponsored internships, I would like to propose > the following project, that I would mentor: > > A music-like audio lavfi source for testing purposes. I believe an audio source that produces something useful would be a very wel

[FFmpeg-devel] [PATCH] avformat: add H264 and HEVC support in IVF muxer

2018-09-30 Thread alx . sukhanov
From: Alex Sukhanov --- libavformat/ivfenc.c | 41 - 1 file changed, 36 insertions(+), 5 deletions(-) diff --git a/libavformat/ivfenc.c b/libavformat/ivfenc.c index 66441a2a43..68c36daf09 100644 --- a/libavformat/ivfenc.c +++ b/libavformat/ivfenc.c @@ -38

[FFmpeg-devel] Mentoring project: music test source

2018-09-30 Thread Nicolas George
Hi. For the next rounds of sponsored internships, I would like to propose the following project, that I would mentor: A music-like audio lavfi source for testing purposes. That means a deterministic pseudo-random stream of notes with varied frequencies, with a structure that looks like music and

Re: [FFmpeg-devel] [PATCH] avfilter/f_cue: use internal fifo for queueing frames

2018-09-30 Thread Nicolas George
Marton Balint (2018-09-30): > With filters.h I can only consume the oldest frame, I have no way to query > the timestamp of the most recent frame in the fifo. Please point me to the > API I should use. If some utility functions you need are missing, do not hesitate to add them. It looks like ff_in

Re: [FFmpeg-devel] [PATCH] avfilter/f_cue: use internal fifo for queueing frames

2018-09-30 Thread Marton Balint
On Sun, 30 Sep 2018, Nicolas George wrote: Marton Balint (2018-09-30): Signed-off-by: Marton Balint --- libavfilter/f_cue.c | 90 +++-- 1 file changed, 32 insertions(+), 58 deletions(-) diff --git a/libavfilter/f_cue.c b/libavfilter/f_cue.c i

Re: [FFmpeg-devel] [PATCH] avfilter/f_cue: use internal fifo for queueing frames

2018-09-30 Thread Nicolas George
Marton Balint (2018-09-30): > Signed-off-by: Marton Balint > --- > libavfilter/f_cue.c | 90 > +++-- > 1 file changed, 32 insertions(+), 58 deletions(-) > > diff --git a/libavfilter/f_cue.c b/libavfilter/f_cue.c > index 732b5e218a..b2fff050da 1006

[FFmpeg-devel] [PATCH] avfilter/f_cue: use internal fifo for queueing frames

2018-09-30 Thread Marton Balint
Signed-off-by: Marton Balint --- libavfilter/f_cue.c | 90 +++-- 1 file changed, 32 insertions(+), 58 deletions(-) diff --git a/libavfilter/f_cue.c b/libavfilter/f_cue.c index 732b5e218a..b2fff050da 100644 --- a/libavfilter/f_cue.c +++ b/libavfilte

[FFmpeg-devel] [PATCH] ffmpeg: break loop when dts_heuristic check done

2018-09-30 Thread xiaofeng
Signed-off-by: xiaofeng -- From 8fd783a0a4e7803e202624ae04e0a38ba18124e3 Mon Sep 17 00:00:00 2001 From: xiaofeng Date: Sun, 30 Sep 2018 20:16:22 +0800 Subject: [PATCH] ffmpeg: break loop when dts_heuristic check done --- fftools/ffmpeg_opt.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletio

[FFmpeg-devel] [PATCH 2/2] lavu/qsv: make a copy as libmfx alignment requirement for uploading

2018-09-30 Thread Zhong Li
Libmfx requires 16 bytes aligned input/output for uploading. Currently only output is 16 byte aligned and assigning same width/height to input with smaller buffer size actually, thus definitely will cause segment fault. Can reproduce with any 1080p nv12 rawvideo input: ffmpeg -init_hw_device qsv=

[FFmpeg-devel] [PATCH 1/2] lavu/qsv: fix a random hwupload failure regression

2018-09-30 Thread Zhong Li
Variable 'ret' hasn't been initialized,thus introducing a random hwupload failure regression due to qsv session uninitialized. Signed-off-by: Zhong Li --- libavutil/hwcontext_qsv.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavutil/hwcontext_qsv.c b/libavutil/hwcontext

Re: [FFmpeg-devel] [PATCH] avcodec: libdav1d AV1 decoder wrapper

2018-09-30 Thread Jean-Baptiste Kempf
On Sat, 29 Sep 2018, at 23:47, Rostislav Pehlivanov wrote: > Given the license difference, the preference of everyone involved with it How can you speak for "everyone involved"? Don't speak for other, please. > for LGPL (its only BSD and a separate library because money) I think It's BSD because