Re: [FFmpeg-devel] [PATCH] avformat:matroskadec: use a define to mark the EBML length is unknown

2019-02-24 Thread Steve Lhomme
On 23/02/2019 19:37, Michael Niedermayer wrote: On Sat, Feb 23, 2019 at 11:14:33AM +0100, Steve Lhomme wrote: From: Steve Lhomme --- libavformat/matroskadec.c | 12 +++- 1 file changed, 7 insertions(+), 5 deletions(-) I think the commit message is a bit terse. This is not just a co

Re: [FFmpeg-devel] [PATCH] avformat:matroskadec: use a define to mark the EBML length is unknown

2019-02-24 Thread Michael Niedermayer
On Sun, Feb 24, 2019 at 09:23:48AM +0100, Steve Lhomme wrote: > On 23/02/2019 19:37, Michael Niedermayer wrote: > >On Sat, Feb 23, 2019 at 11:14:33AM +0100, Steve Lhomme wrote: > >>From: Steve Lhomme > >> > >>--- > >> libavformat/matroskadec.c | 12 +++- > >> 1 file changed, 7 insertions(

Re: [FFmpeg-devel] [PATCH 1/2] avformat/mpegtsenc: factorize writing registration_descriptor

2019-02-24 Thread Carl Eugen Hoyos
2019-02-23 20:36 GMT+01:00, Marton Balint : > Signed-off-by: Marton Balint > --- > libavformat/mpegtsenc.c | 50 > ++--- > 1 file changed, 18 insertions(+), 32 deletions(-) > > diff --git a/libavformat/mpegtsenc.c b/libavformat/mpegtsenc.c > index 35a72

Re: [FFmpeg-devel] avcodec/qtrle : improve decoding speed of 24bpp and 32 bpp

2019-02-24 Thread Carl Eugen Hoyos
2019-02-23 16:02 GMT+01:00, Martin Vignali : > Hello, > > Patch in attach, improve decoding speed of qtrle (also known as Mov > Animation) > > Can't test on big endian. Test on big endian is welcome. Patch 1 works on BE and lgtm. Patch 2 does not work, correct change for case 32 on le and be is:

Re: [FFmpeg-devel] fate/proresenc_aw : Add fate test for interlace and 444 encoding

2019-02-24 Thread Michael Niedermayer
On Sat, Feb 23, 2019 at 07:10:06PM +0100, Martin Vignali wrote: > Hello, > > Patch in attach add fate test for interlace and 444 encoding > > All prores_aw encoding test can be test with : > > make fate-vsynth3-prores;make fate-vsynth2-prores;make > fate-vsynth1-prores;make fate-vsynth_lena-pror

Re: [FFmpeg-devel] [PATCH 1/3] avcodec/avpacket: add some assertions to ensure pkt->data is not null if pkt->size > 0

2019-02-24 Thread Marton Balint
On Sun, 17 Feb 2019, Marton Balint wrote: This should fix the following Coverity false positives: Coverity CID #1405450. Coverity CID #1430930. Signed-off-by: Marton Balint Ping for the series, will apply soon. Regards, Marton ___ ffmpeg-devel

Re: [FFmpeg-devel] [PATCH 2/4] avformat/utils: be more strict about stream specifiers

2019-02-24 Thread Marton Balint
On Tue, 19 Feb 2019, Carl Eugen Hoyos wrote: 2019-02-18 22:43 GMT+01:00, Marton Balint : On Sun, 17 Feb 2019, Carl Eugen Hoyos wrote: 2019-02-17 20:55 GMT+01:00, Marton Balint : This reworks the code to be more strict about accepting stream specifiers. From now on we strictly enforce the

Re: [FFmpeg-devel] [PATCH 2/2] avformat/mpegtsenc: write format_identifier HEVC for HEVC streams

2019-02-24 Thread Marton Balint
On Sat, 23 Feb 2019, Carl Eugen Hoyos wrote: Hi Marton! Am 23.02.2019 um 20:36 schrieb Marton Balint : This improves compatibility with some consumer TVs which apparently either search a HEVC descriptor (which our mpegts muxer can't generate) or a format specifier. Since the HEVC format sp

Re: [FFmpeg-devel] [PATCH 2/2] avformat/mpegtsenc: write format_identifier HEVC for HEVC streams

2019-02-24 Thread Carl Eugen Hoyos
> Am 24.02.2019 um 23:39 schrieb Marton Balint : > >> On Sat, 23 Feb 2019, Carl Eugen Hoyos wrote: >> >> Hi Marton! >> >>> Am 23.02.2019 um 20:36 schrieb Marton Balint : >>> This improves compatibility with some consumer TVs which apparently either >>> search a HEVC descriptor (which our mpeg

[FFmpeg-devel] [PATCH 2/2] avcodec/gdv: Check for truncated tags in decompress_5()

2019-02-24 Thread Michael Niedermayer
Testcase: 13169/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_GDV_fuzzer-5666354038833152 Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg Signed-off-by: Michael Niedermayer --- libavcodec/gdv.c | 2 ++ 1 file changed, 2 insertions(+) dif

[FFmpeg-devel] [PATCH 1/2] avcodec/bethsoftvideo: Check block_type

2019-02-24 Thread Michael Niedermayer
Fixes: Timeout (17 seconds -> 1 second) Fixes: 13184/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_BETHSOFTVID_fuzzer-5711446296494080 Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg Signed-off-by: Michael Niedermayer --- libavcodec/beths

Re: [FFmpeg-devel] [PATCH 1/2] avformat/dashenc: Added option to repeatedly publish master playlist

2019-02-24 Thread Jeyapal, Karthick
On 2/19/19 12:18 PM, Karthick J wrote: > The master playlist can be published at a specified interval with this option > --- > doc/muxers.texi | 3 +++ > libavformat/dashenc.c | 9 - > 2 files changed, 11 insertions(+), 1 deletion(-) > > diff --git a/doc/muxers.texi b/doc/muxers.tex