[FFmpeg-devel] [PATCH V2 1/2] configure: sort decoder/encoder/filter/... names in alphabet order

2019-04-14 Thread Guo, Yejun
take decoder names an example, with the default page length, shell command 'pr' needs two pages for all the decoder names. The names are firstly printed in the first page, then in the second page. So, as a whole, the names are sorted neither in column order nor in row order. It's a little

[FFmpeg-devel] [PATCH V2 2/2] configure: replace 'pr' with printf since busybox does not support pr

2019-04-14 Thread Guo, Yejun
It is part of change from https://trac.ffmpeg.org/ticket/5680 provided by Kylie McClain at Wed, 29 Jun 2016 16:37:20 -0400. That change contains two parts, in function log_file and in function print_in_columns. The second part is not good, so I have send out a new patch for print_in_columns.

[FFmpeg-devel] [PATCH, v3] lavu/hwcontext_qsv: Fix the realign check for hwupload

2019-04-14 Thread Linjie Fu
Fix the aligned check in hwupload, input surface should be 16 aligned too. Fix #7830. Signed-off-by: Linjie Fu --- libavutil/hwcontext_qsv.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/libavutil/hwcontext_qsv.c b/libavutil/hwcontext_qsv.c index b6d8bfe2bf..197dd8002a

[FFmpeg-devel] [PATCH, v2] lavc/qsvenc: Fix the memory leak for enc_ctrl.Payload

2019-04-14 Thread Linjie Fu
frame->enc_ctrl.Payload is malloced in get_free_frame, directly memset the whole structure of enc_ctrl to zero will cause the memory leak for enc_ctrl.Payload. frame->enc_ctrl as a structure will be malloc and init to zero by calling frame = av_mallocz(sizeof(*frame)), so the memset is redundant

[FFmpeg-devel] [PATCH] lavf/qsvvpp: avoid the double-free when working in sys memory mode

2019-04-14 Thread Linjie Fu
Currently, picref will be freed by calling av_frame_free() in submit_frame() in qsvvpp.c when working in system memory mode,and normally it is freed in filter_frame() in vf_vpp_qsv.c when working in other modes. Double free happens when working in system memory mode, remove to fix the memory

Re: [FFmpeg-devel] [PATCH] configure: sort decoder/encoder/filter/... names in alphabet order, row by row

2019-04-14 Thread Guo, Yejun
> -Original Message- > From: Guo, Yejun > Sent: Friday, April 12, 2019 9:24 PM > To: ffmpeg-devel@ffmpeg.org > Cc: Guo, Yejun > Subject: [PATCH] configure: sort decoder/encoder/filter/... names in > alphabet order, row by row > > take decoder names an example, with the default page

Re: [FFmpeg-devel] [PATCH v4 1/7] vf_crop: Add support for cropping hardware frames

2019-04-14 Thread Song, Ruiling
> -Original Message- > From: ffmpeg-devel [mailto:ffmpeg-devel-boun...@ffmpeg.org] On Behalf Of > Mark Thompson > Sent: Wednesday, April 10, 2019 6:07 AM > To: ffmpeg-devel@ffmpeg.org > Subject: [FFmpeg-devel] [PATCH v4 1/7] vf_crop: Add support for cropping > hardware frames > > Set

Re: [FFmpeg-devel] [PATCH v4 2/7] doc/indevs: Add example using cropping to capture part of a plane

2019-04-14 Thread Song, Ruiling
> -Original Message- > From: ffmpeg-devel [mailto:ffmpeg-devel-boun...@ffmpeg.org] On Behalf Of > Mark Thompson > Sent: Wednesday, April 10, 2019 6:07 AM > To: ffmpeg-devel@ffmpeg.org > Subject: [FFmpeg-devel] [PATCH v4 2/7] doc/indevs: Add example using cropping > to capture part of a

Re: [FFmpeg-devel] [PATCH] fftools/ffmpeg_filter: Fix error when both hwaccel and stream_loop applied

2019-04-14 Thread myp...@gmail.com
On Mon, Apr 15, 2019 at 11:11 AM Leo Zhang wrote: > > For example: > ./ffmpeg -hwaccel cuvid -c:v h264_cuvid -stream_loop -1 -i in.flv -c:v > h264_nvenc out.flv > will print below error messages after one loop: > Impossible to convert between the formats supported by the filter >

Re: [FFmpeg-devel] [PATCH V1 1/2] lavf/hls: Cleanup the applehttp

2019-04-14 Thread myp...@gmail.com
On Fri, Apr 12, 2019 at 10:08 PM Steven Liu wrote: > > > > > 在 2019年4月12日,19:20,Jun Zhao 写道: > > > > From: Jun Zhao > > > > Cleanup the applehttp as demuxer name, when use the command : > > > > ffmpeg -formats, get the confused information like: > > " > > E hls Apple HTTP Live

[FFmpeg-devel] [PATCH V1] lavf/oggparsevorbis: Fix change the case of metadata keys issue

2019-04-14 Thread Jun Zhao
From: Jun Zhao The spec in https://xiph.org/vorbis/doc/v-comment.html states that the metadata keys are case-insensitive, so don't change the case and update the fate test case. Fix #7784 Signed-off-by: Jun Zhao --- libavformat/oggparsevorbis.c |9 -

[FFmpeg-devel] [PATCH] avcodec/dxtory: Check slice_size against minimum in dxtory_decode_v2()

2019-04-14 Thread Michael Niedermayer
Fixes: Timeout (33sec -> 16 milli sec) Fixes: 14181/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_DXTORY_fuzzer-5681840708386816 Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg Signed-off-by: Michael Niedermayer --- libavcodec/dxtory.c |

[FFmpeg-devel] [PATCH] avcodec/cbs_av1: add support for Scalability Metadata

2019-04-14 Thread James Almer
Signed-off-by: James Almer --- This will make the AV1RawObu struct weigh about ~3kb instead of ~1kb. libavcodec/av1.h | 33 libavcodec/cbs_av1.h | 15 +- libavcodec/cbs_av1_syntax_template.c | 45 ++-- 3

Re: [FFmpeg-devel] [PATCH] avformat/mxfdec: use operational_pattern_ul instead of operational_pattern for metadata

2019-04-14 Thread Marton Balint
On Sun, 14 Apr 2019, Tomas Härdin wrote: sön 2019-04-14 klockan 18:51 +0200 skrev Marton Balint: This makes it more consistent with other metadata keys. Signed-off-by: Marton Balint ---  libavformat/mxfdec.c   | 2 +-  tests/ref/fate/mxf-probe-d10   | 2 +-  

[FFmpeg-devel] [PATCH]lavfi/fspp: Remove two unused macros

2019-04-14 Thread Carl Eugen Hoyos
Hi! Attached patch removes two unused macros as a preparation for a new patch to silence a clang warning. Please comment, Carl Eugen From 133f21a9df22f5ae15ea7feef57bb6c1bc7a7bf4 Mon Sep 17 00:00:00 2001 From: Carl Eugen Hoyos Date: Sun, 14 Apr 2019 22:56:17 +0200 Subject: [PATCH] lavfi/fspp:

Re: [FFmpeg-devel] [PATCH 2/2 v2] avcodec/cbs_av1: add support for Padding OBUs

2019-04-14 Thread James Almer
On 4/14/2019 2:22 PM, Mark Thompson wrote: > On 13/04/2019 20:25, James Almer wrote: >> Based on itut_t35 Matadata OBU parsing code. >> >> Signed-off-by: James Almer >> --- >> libavcodec/cbs_av1.c | 20 >> libavcodec/cbs_av1.h | 7 +++ >>

Re: [FFmpeg-devel] [PATCH 1/2] avcodec/cbs_av1: add a function to get a payload size without trailing zero bytes

2019-04-14 Thread James Almer
On 4/14/2019 2:17 PM, Mark Thompson wrote: > On 13/04/2019 20:21, James Almer wrote: >> Factor it out from cbs_av1_read_metadata_itut_t35() >> >> Signed-off-by: James Almer >> --- >> libavcodec/cbs_av1.c | 11 +++ >> libavcodec/cbs_av1_syntax_template.c | 10 +- >>

Re: [FFmpeg-devel] [PATCH] avcodec: add drop_changed_frames

2019-04-14 Thread James Almer
On 4/14/2019 3:29 PM, Hendrik Leppkes wrote: > On Sun, Apr 14, 2019 at 6:50 PM Gyan wrote: >> >> Implemented this patch >> http://www.ffmpeg.org/pipermail/ffmpeg-devel/2019-March/241733.html >> >> >> in libavcodec as suggested by Michael >> > > This sure adds a lot of additional fields to the

Re: [FFmpeg-devel] [PATCH] avcodec: add drop_changed_frames

2019-04-14 Thread Hendrik Leppkes
On Sun, Apr 14, 2019 at 6:50 PM Gyan wrote: > > Implemented this patch > http://www.ffmpeg.org/pipermail/ffmpeg-devel/2019-March/241733.html > > > in libavcodec as suggested by Michael > This sure adds a lot of additional fields to the main struct for a rather specialized feature, that I

Re: [FFmpeg-devel] [PATCH] avformat/mxfdec: use operational_pattern_ul instead of operational_pattern for metadata

2019-04-14 Thread Tomas Härdin
sön 2019-04-14 klockan 18:51 +0200 skrev Marton Balint: > This makes it more consistent with other metadata keys. > > Signed-off-by: Marton Balint > --- >  libavformat/mxfdec.c   | 2 +- >  tests/ref/fate/mxf-probe-d10   | 2 +- >  tests/ref/fate/mxf-probe-dnxhd | 2 +- >  

Re: [FFmpeg-devel] [PATCH] avcodec: add drop_changed_frames

2019-04-14 Thread Marton Balint
On Sun, 14 Apr 2019, Gyan wrote: On 14-04-2019 10:29 PM, Marton Balint wrote: On Sun, 14 Apr 2019, Gyan wrote: Implemented this patch http://www.ffmpeg.org/pipermail/ffmpeg-devel/2019-March/241733.html  in libavcodec as suggested by Michael Docs should mention that you are not

Re: [FFmpeg-devel] [PATCH] avcodec: add drop_changed_frames

2019-04-14 Thread Gyan
On 14-04-2019 10:29 PM, Marton Balint wrote: On Sun, 14 Apr 2019, Gyan wrote: Implemented this patch http://www.ffmpeg.org/pipermail/ffmpeg-devel/2019-March/241733.html  in libavcodec as suggested by Michael Docs should mention that you are not only dropping the frames but returning

Re: [FFmpeg-devel] [PATCH 3/5] avformat/mxfdec: guess wrapping of tracks by other tracks with the same body sid

2019-04-14 Thread Marton Balint
On Sun, 14 Apr 2019, Tomas Härdin wrote: fre 2019-04-12 klockan 01:09 +0200 skrev Marton Balint: This affects the following samples: samples/ffmpeg-bugs/roundup/issue1775/av_seek_frame_failure.mxf samples/ffmpeg-bugs/trac/ticket1957/16ch.mxf samples/ffmpeg-bugs/trac/ticket5016/r0.mxf

Re: [FFmpeg-devel] [PATCH 2/2 v2] avcodec/cbs_av1: add support for Padding OBUs

2019-04-14 Thread Mark Thompson
On 13/04/2019 20:25, James Almer wrote: > Based on itut_t35 Matadata OBU parsing code. > > Signed-off-by: James Almer > --- > libavcodec/cbs_av1.c | 20 > libavcodec/cbs_av1.h | 7 +++ > libavcodec/cbs_av1_syntax_template.c | 24

Re: [FFmpeg-devel] [PATCH 1/2] avcodec/cbs_av1: add a function to get a payload size without trailing zero bytes

2019-04-14 Thread Mark Thompson
On 13/04/2019 20:21, James Almer wrote: > Factor it out from cbs_av1_read_metadata_itut_t35() > > Signed-off-by: James Almer > --- > libavcodec/cbs_av1.c | 11 +++ > libavcodec/cbs_av1_syntax_template.c | 10 +- > 2 files changed, 12 insertions(+), 9 deletions(-)

Re: [FFmpeg-devel] [PATCH 3/5] avformat/mxfdec: guess wrapping of tracks by other tracks with the same body sid

2019-04-14 Thread Baptiste Coudurier
Hi Tomas, I hope you are doing well > On Apr 14, 2019, at 8:58 AM, Tomas Härdin wrote: > > […] > >> @@ -2553,6 +2563,12 @@ static int mxf_parse_structural_metadata(MXFContext >> *mxf) >> } >> } >> >> +for (int i = 0; i < mxf->fc->nb_streams; i++) { >> +MXFTrack

Re: [FFmpeg-devel] [PATCH] avcodec: add drop_changed_frames

2019-04-14 Thread Marton Balint
On Sun, 14 Apr 2019, Gyan wrote: Implemented this patch http://www.ffmpeg.org/pipermail/ffmpeg-devel/2019-March/241733.html  in libavcodec as suggested by Michael Docs should mention that you are not only dropping the frames but returning error as well. (I agree that error should be

[FFmpeg-devel] [PATCH] avformat/mxfdec: use operational_pattern_ul instead of operational_pattern for metadata

2019-04-14 Thread Marton Balint
This makes it more consistent with other metadata keys. Signed-off-by: Marton Balint --- libavformat/mxfdec.c | 2 +- tests/ref/fate/mxf-probe-d10 | 2 +- tests/ref/fate/mxf-probe-dnxhd | 2 +- tests/ref/fate/mxf-probe-dv25 | 2 +- 4 files changed, 4 insertions(+), 4 deletions(-)

[FFmpeg-devel] [PATCH] avcodec: add drop_changed_frames

2019-04-14 Thread Gyan
Implemented this patch http://www.ffmpeg.org/pipermail/ffmpeg-devel/2019-March/241733.html  in libavcodec as suggested by Michael Gyan P.S. will add doc/APIchanges entry after I have the hash. From bbc8ac18da560a59cd252bbb5c34dfa1fcc575b8 Mon Sep 17 00:00:00 2001 From: Gyan Doshi Date: Sun,

Re: [FFmpeg-devel] [PATCH 5/5] avformat/mxfdec: fix and enhance RIP KLV length checks

2019-04-14 Thread Tomas Härdin
fre 2019-04-12 klockan 01:09 +0200 skrev Marton Balint: > KLV length is BER encoded (variable size), but the code assumed the encoding > to > always use 4 bytes. > > Fixes parsing Random Index Pack in > samples/MXF/issue2160/PW0805A0V01.4C5B5636.EFA330.mxf. > > > Signed-off-by: Marton Balint

Re: [FFmpeg-devel] [PATCH 3/5] avformat/mxfdec: guess wrapping of tracks by other tracks with the same body sid

2019-04-14 Thread Tomas Härdin
fre 2019-04-12 klockan 01:09 +0200 skrev Marton Balint: > This affects the following samples: > > samples/ffmpeg-bugs/roundup/issue1775/av_seek_frame_failure.mxf > samples/ffmpeg-bugs/trac/ticket1957/16ch.mxf > samples/ffmpeg-bugs/trac/ticket5016/r0.mxf >

Re: [FFmpeg-devel] [PATCH 4/5] avformat/mxfdec: take into account run-in in find_partition_by_offset

2019-04-14 Thread Tomas Härdin
fre 2019-04-12 klockan 01:09 +0200 skrev Marton Balint: > > Signed-off-by: Marton Balint > --- >  libavformat/mxfdec.c | 8 >  1 file changed, 4 insertions(+), 4 deletions(-) > > diff --git a/libavformat/mxfdec.c b/libavformat/mxfdec.c > index 236294880e..6f0f87763d 100644 > ---

Re: [FFmpeg-devel] [PATCH 2/5] avformat/mxfdec: rework mxf_essence_container_end

2019-04-14 Thread Tomas Härdin
fre 2019-04-12 klockan 01:09 +0200 skrev Marton Balint: > We find the last essence container much faster if we go through the partitions > backwards... Good catch > > > Signed-off-by: Marton Balint > --- >  libavformat/mxfdec.c | 9 +++-- >  1 file changed, 3 insertions(+), 6 deletions(-) >

Re: [FFmpeg-devel] [PATCH 1/5] avformat/mxfdec: pass track to mxf_edit_unit_absolute_offset instead of arbitrary edit unit

2019-04-14 Thread Tomas Härdin
fre 2019-04-12 klockan 01:09 +0200 skrev Marton Balint: > > Signed-off-by: Marton Balint > --- >  libavformat/mxfdec.c | 12 ++-- >  1 file changed, 6 insertions(+), 6 deletions(-) > > diff --git a/libavformat/mxfdec.c b/libavformat/mxfdec.c > index 8c65a2bbcf..c3892a1037 100644 > ---

Re: [FFmpeg-devel] [PATCH] avformat/mxfdec: export operational pattern UL as file metadata

2019-04-14 Thread Tomas Härdin
tor 2019-04-11 klockan 22:20 +0200 skrev Marton Balint: > > The metadata tag name became "operational_pattern", I wonder if it might  > make more sense to use "operational_pattern_ul" instead, it feels a little  > more consistent with the other custom mxf metadata entries. > > It is not too late

[FFmpeg-devel] [PATCH] tests/fate: add fate-hls-segment-size for hls_segment_size test

2019-04-14 Thread Steven Liu
Signed-off-by: Steven Liu --- tests/fate/hlsenc.mak | 11 +++ 1 file changed, 11 insertions(+) diff --git a/tests/fate/hlsenc.mak b/tests/fate/hlsenc.mak index 80536239fc..489d47b888 100644 --- a/tests/fate/hlsenc.mak +++ b/tests/fate/hlsenc.mak @@ -41,3 +41,14 @@ fate-hls-live-endlist:

Re: [FFmpeg-devel] [PATCH] lavfi: add nlmeans_opencl filter

2019-04-14 Thread Song, Ruiling
> -Original Message- > From: ffmpeg-devel [mailto:ffmpeg-devel-boun...@ffmpeg.org] On Behalf Of > Mark Thompson > Sent: Sunday, April 14, 2019 1:23 AM > To: ffmpeg-devel@ffmpeg.org > Subject: Re: [FFmpeg-devel] [PATCH] lavfi: add nlmeans_opencl filter > > On 12/04/2019 08:38, Song,

[FFmpeg-devel] [PATCH 2/2] avcodec/arbc: Try to correct keyframe/frame type

2019-04-14 Thread Michael Niedermayer
Signed-off-by: Michael Niedermayer --- libavcodec/arbc.c | 31 +++ 1 file changed, 19 insertions(+), 12 deletions(-) diff --git a/libavcodec/arbc.c b/libavcodec/arbc.c index 08d3a0ae6b..bce4a4c96d 100644 --- a/libavcodec/arbc.c +++ b/libavcodec/arbc.c @@ -38,15

[FFmpeg-devel] [PATCH 1/2] avcodec/arbc: Skip unchanged frames

2019-04-14 Thread Michael Niedermayer
Fixes: Timeout (16sec -> 5sec) Fixes: 14128/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_ARBC_fuzzer-5767365721063424 Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg Signed-off-by: Michael Niedermayer --- libavcodec/arbc.c | 2 +- 1