Re: [FFmpeg-devel] [PATCH]lavf/matroska: Support codec id V_FFV1 for FFV1.

2017-02-28 Thread Michael Niedermayer
On Tue, Feb 28, 2017 at 08:45:44PM +0100, Carl Eugen Hoyos wrote: > Hi! > > Attached patch fixes ticket #6206 here. > > Please comment, Carl Eugen previous ffmpeg versions dont play these files [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB While the State

[FFmpeg-devel] [PATCH] avcodec/vp56: Clear dimensions in case of failure in the middle of a resolution change

2017-02-28 Thread Michael Niedermayer
Similar code is used elsewhere in vp56 to force a more complete reinit in the future. Fixes null pointer dereference Fixes: 707/clusterfuzz-testcase-4717453097566208 Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/targets/ffmpeg Signed-off-by: Michael

Re: [FFmpeg-devel] [PATCH 2/2] avcodec/vp8: Check for bitsteam end in decode_mb_row_no_filter()

2017-02-28 Thread Ronald S. Bultje
Hi Michael, On Tue, Feb 28, 2017 at 11:29 AM, Michael Niedermayer < mich...@niedermayer.cc> wrote: > On Tue, Feb 28, 2017 at 07:44:12AM -0500, Ronald S. Bultje wrote: > > Hi, > > > > On Mon, Feb 27, 2017 at 10:28 PM, Michael Niedermayer < > > mich...@niedermayer.cc> wrote: > > > > > Fixes:

[FFmpeg-devel] [PATCH] avformat/hlsenc: move the segment files handle close before temp flags process

2017-02-28 Thread Steven Liu
fix ticket: #6204 Signed-off-by: Steven Liu --- libavformat/hlsenc.c | 7 --- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/libavformat/hlsenc.c b/libavformat/hlsenc.c index 9cf6211..b8122f1 100644 --- a/libavformat/hlsenc.c +++ b/libavformat/hlsenc.c

Re: [FFmpeg-devel] [PATCH]Makefile: Clean compat subdirectory

2017-02-28 Thread Michael Niedermayer
On Tue, Feb 21, 2017 at 02:38:22PM +0100, Carl Eugen Hoyos wrote: > Hi! > > Attached patch fixes ticket #5546 here. > > Please comment, Carl Eugen > Makefile |2 ++ > 1 file changed, 2 insertions(+) > 47464e0891035a702d5228a79e40b4855e6262a1 >

Re: [FFmpeg-devel] [PATCH v3 2/2] rtp: rfc4175: add handler for YCbCr-4:2:2

2017-02-28 Thread Thomas Volkert
On 28.02.2017 19:39, Damien Riegel wrote: > This adds partial support for the RFC 4175 (raw video over RTP). The > only supported formats are the YCbCr-4:2:2 8 bit because it's natively > supported by FFmpeg with pixel format UYVY, and 10 bit which requires > the vrawdepay codec to convert the

Re: [FFmpeg-devel] [PATCH v3 2/2] rtp: rfc4175: add handler for YCbCr-4:2:2

2017-02-28 Thread Hendrik Leppkes
On Tue, Feb 28, 2017 at 11:40 PM, Rostislav Pehlivanov wrote: >> +static int rfc4175_parse_format(AVStream *stream, PayloadContext *data) >> +{ >> +enum AVPixelFormat pixfmt = AV_PIX_FMT_NONE; >> +int bits_per_sample = 0; >> +int tag = 0; >> + >> +if

Re: [FFmpeg-devel] [PATCH v3 2/2] rtp: rfc4175: add handler for YCbCr-4:2:2

2017-02-28 Thread Rostislav Pehlivanov
On 28 February 2017 at 18:39, Damien Riegel < damien.rie...@savoirfairelinux.com> wrote: > This adds partial support for the RFC 4175 (raw video over RTP). The > only supported formats are the YCbCr-4:2:2 8 bit because it's natively > supported by FFmpeg with pixel format UYVY, and 10 bit which

Re: [FFmpeg-devel] [PATCH v3 1/2] codec: bitpacked: add decoder

2017-02-28 Thread Michael Niedermayer
On Tue, Feb 28, 2017 at 01:39:07PM -0500, Damien Riegel wrote: > Add a codec capable of decoding some formats of the RFC4175. For now > it's only capable of handling YCbCr-4:2:2 with 8-bit or 10-bit depth. > > For 8-bit it's a simple pass-through, for 10-bit it depacks the stream > in the

Re: [FFmpeg-devel] [PATCH] vf_hwupload: Add missing return value check

2017-02-28 Thread Mark Thompson
On 27/02/17 02:23, Jun Zhao wrote: > From e60a958c78ba084e0745acebec5125394387c5b1 Mon Sep 17 00:00:00 2001 > From: Jun Zhao > Date: Mon, 27 Feb 2017 10:08:36 +0800 > Subject: [PATCH] vf_hwupload: Add missing return value checks > > Add missing return value checks to fix the

[FFmpeg-devel] [PATCH 3/3] avcodec/wavpack: Check value before shift in wp_exp2()

2017-02-28 Thread Michael Niedermayer
Fixes undefined shift, all callers should be changed to check the value they use with wp_exp2() or its return value. Fixes: 692/clusterfuzz-testcase-5757381516460032 Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/targets/ffmpeg Signed-off-by: Michael

[FFmpeg-devel] [PATCH 2/3] avcodec/ituh263dec: Use 0xffff as error code in h263p_decode_umotion()

2017-02-28 Thread Michael Niedermayer
This matches ff_h263_decode_motion() both functions error codes are interpreted by the same common code Fixes: 690/clusterfuzz-testcase-4744944981901312 Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/targets/ffmpeg Signed-off-by: Michael Niedermayer

[FFmpeg-devel] [PATCH 1/3] avcodec/h264idct_template: Fix several runtime error: signed integer overflow

2017-02-28 Thread Michael Niedermayer
Fixes: 689/clusterfuzz-testcase-6029352737177600 Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/targets/ffmpeg Signed-off-by: Michael Niedermayer --- libavcodec/h264idct_template.c | 18 +- 1 file changed, 9

[FFmpeg-devel] [PATCH]lavf/matroska: Support codec id V_FFV1 for FFV1.

2017-02-28 Thread Carl Eugen Hoyos
Hi! Attached patch fixes ticket #6206 here. Please comment, Carl Eugen From 10caa3898fa76f5a76dce7e3795a7a0d475870c1 Mon Sep 17 00:00:00 2001 From: Carl Eugen Hoyos Date: Tue, 28 Feb 2017 20:36:12 +0100 Subject: [PATCH] lavf/matroska: Support new codec id V_FFV1 for FFV1.

Re: [FFmpeg-devel] [PATCHv3 3/3] mkv: Export bounds and padding from spherical metadata

2017-02-28 Thread James Almer
On 2/28/2017 3:21 PM, Vittorio Giovara wrote: > On Tue, Feb 28, 2017 at 1:15 PM, James Almer wrote: >> On 2/28/2017 2:46 PM, Vittorio Giovara wrote: >>> On Tue, Feb 28, 2017 at 11:00 AM, James Almer wrote: On 2/22/2017 1:21 PM, James Almer wrote: >

[FFmpeg-devel] Hardware Encode H264

2017-02-28 Thread ALEXANDRE MORO
Hello everyone, Im trying both encode/decode h264 stream for live streaming. I really dont know how to use the hardware encoder cuda nor nvenc nor dxva2 and i do not find any documentation on that. 1) I already compiled ffmpeg 2017-02-28 snapshot) with --enable-cuda \ --enable-cuvid \

[FFmpeg-devel] [PATCH v3 1/2] codec: bitpacked: add decoder

2017-02-28 Thread Damien Riegel
Add a codec capable of decoding some formats of the RFC4175. For now it's only capable of handling YCbCr-4:2:2 with 8-bit or 10-bit depth. For 8-bit it's a simple pass-through, for 10-bit it depacks the stream in the AV_PIX_FMT_YUV422P10 pixel format. Signed-off-by: Damien Riegel

[FFmpeg-devel] [PATCH v3 2/2] rtp: rfc4175: add handler for YCbCr-4:2:2

2017-02-28 Thread Damien Riegel
This adds partial support for the RFC 4175 (raw video over RTP). The only supported formats are the YCbCr-4:2:2 8 bit because it's natively supported by FFmpeg with pixel format UYVY, and 10 bit which requires the vrawdepay codec to convert the payload in a format handled by FFmpeg.

Re: [FFmpeg-devel] [PATCHv3 3/3] mkv: Export bounds and padding from spherical metadata

2017-02-28 Thread Vittorio Giovara
On Tue, Feb 28, 2017 at 1:15 PM, James Almer wrote: > On 2/28/2017 2:46 PM, Vittorio Giovara wrote: >> On Tue, Feb 28, 2017 at 11:00 AM, James Almer wrote: >>> On 2/22/2017 1:21 PM, James Almer wrote: On 2/21/2017 7:35 PM, Vittorio Giovara wrote: >>>

Re: [FFmpeg-devel] [PATCHv3 3/3] mkv: Export bounds and padding from spherical metadata

2017-02-28 Thread James Almer
On 2/28/2017 2:46 PM, Vittorio Giovara wrote: > On Tue, Feb 28, 2017 at 11:00 AM, James Almer wrote: >> On 2/22/2017 1:21 PM, James Almer wrote: >>> On 2/21/2017 7:35 PM, Vittorio Giovara wrote: >> >> CCing this one as well. >> --- libavformat/matroskadec.c

Re: [FFmpeg-devel] [PATCHv3 3/3] mkv: Export bounds and padding from spherical metadata

2017-02-28 Thread Vittorio Giovara
On Tue, Feb 28, 2017 at 12:46 PM, Vittorio Giovara wrote: >> >> I think this'll look better as >> >> >> case MATROSKA_VIDEO_PROJECTION_TYPE_EQUIRECTANGULAR: >> projection = AV_SPHERICAL_EQUIRECTANGULAR; >> >> if

Re: [FFmpeg-devel] [PATCHv3 3/3] mkv: Export bounds and padding from spherical metadata

2017-02-28 Thread Vittorio Giovara
On Tue, Feb 28, 2017 at 11:00 AM, James Almer wrote: > On 2/22/2017 1:21 PM, James Almer wrote: >> On 2/21/2017 7:35 PM, Vittorio Giovara wrote: > > CCing this one as well. > >>> --- >>> libavformat/matroskadec.c | 64 >>> -- >>>

Re: [FFmpeg-devel] [PATCHv3 2/3] mov: Export bounds and padding from spherical metadata

2017-02-28 Thread Vittorio Giovara
On Tue, Feb 28, 2017 at 10:58 AM, James Almer wrote: > On 2/21/2017 8:07 PM, James Almer wrote: >> On 2/21/2017 7:35 PM, Vittorio Giovara wrote: >>> Update the fate test as needed. >>> --- >>> libavformat/mov.c | 28 +++- >>>

Re: [FFmpeg-devel] [PATCH 2/2] avcodec/vp8: Check for bitsteam end in decode_mb_row_no_filter()

2017-02-28 Thread Michael Niedermayer
On Tue, Feb 28, 2017 at 07:44:12AM -0500, Ronald S. Bultje wrote: > Hi, > > On Mon, Feb 27, 2017 at 10:28 PM, Michael Niedermayer < > mich...@niedermayer.cc> wrote: > > > Fixes: 686/clusterfuzz-testcase-5853946876788736 > > > > Found-by: continuous fuzzing process https://github.com/google/oss-

Re: [FFmpeg-devel] [PATCHv3 2/3] mov: Export bounds and padding from spherical metadata

2017-02-28 Thread James Almer
On 2/21/2017 8:07 PM, James Almer wrote: > On 2/21/2017 7:35 PM, Vittorio Giovara wrote: >> Update the fate test as needed. >> --- >> libavformat/mov.c | 28 +++- >> tests/ref/fate/mov-spherical-mono | 6 +- >> 2 files changed, 32 insertions(+), 2

Re: [FFmpeg-devel] [PATCHv3 3/3] mkv: Export bounds and padding from spherical metadata

2017-02-28 Thread James Almer
On 2/22/2017 1:21 PM, James Almer wrote: > On 2/21/2017 7:35 PM, Vittorio Giovara wrote: CCing this one as well. >> --- >> libavformat/matroskadec.c | 64 >> -- >> tests/ref/fate/matroska-spherical-mono | 6 +++- >> 2 files changed, 66

Re: [FFmpeg-devel] [PATCH] Remove qmin and qmax constraints for nvenc vbr

2017-02-28 Thread Timo Rothenpieler
> Patch looks good to me too. Timo, I can push it if you don't mind not > testing it yourself. I have some stilistic nits, otherwise it's ok. It leaves the fall through comment dangling, even though there is no fall through anymore. And it could use fallthrough to in the new location of

[FFmpeg-devel] [PATCH 3/3] avformat/oggdec: fix leak in ogg_restore()

2017-02-28 Thread Michael Niedermayer
Fixes: asan_bug_leak Found-by: Thomas Guilbert Signed-off-by: Michael Niedermayer --- libavformat/oggdec.c | 6 +- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/libavformat/oggdec.c b/libavformat/oggdec.c index

[FFmpeg-devel] [PATCH 1/3] avformat/oggdec: remove unused parameter of ogg_restore()

2017-02-28 Thread Michael Niedermayer
Signed-off-by: Michael Niedermayer --- libavformat/oggdec.c | 13 + 1 file changed, 5 insertions(+), 8 deletions(-) diff --git a/libavformat/oggdec.c b/libavformat/oggdec.c index e1ef21337c..64a88261e3 100644 --- a/libavformat/oggdec.c +++

[FFmpeg-devel] [PATCH 2/3] avformat/oggdec: Factor free_stream out

2017-02-28 Thread Michael Niedermayer
Signed-off-by: Michael Niedermayer --- libavformat/oggdec.c | 23 --- 1 file changed, 16 insertions(+), 7 deletions(-) diff --git a/libavformat/oggdec.c b/libavformat/oggdec.c index 64a88261e3..98cd0f55c7 100644 --- a/libavformat/oggdec.c +++

Re: [FFmpeg-devel] [PATCH] Remove qmin and qmax constraints for nvenc vbr

2017-02-28 Thread Philip Langdale
On Tue, 28 Feb 2017 11:54:32 +0100 Miroslav Slugeň wrote: > Dne 28.2.2017 v 10:44 Timo Rothenpieler napsal(a): > > Am 28.02.2017 um 01:24 schrieb Ganapathy Raman Kasi: > >> Can someone please help review patch. Thanks. > > I have seen an marked it, but I don't have time

Re: [FFmpeg-devel] [ffmpeg-devel] [PATCH] nvenc: add initial QP value options for nvenc

2017-02-28 Thread Philip Langdale
On Tue, 28 Feb 2017 05:44:04 + Konda Raju wrote: > Hi, > > The attached patch adds initial qp value options for I, P and B > frames. > > -Raju Konda > > --- > This email message is for the

Re: [FFmpeg-devel] [PATCH 2/2] avcodec/vp8: Check for bitsteam end in decode_mb_row_no_filter()

2017-02-28 Thread Ronald S. Bultje
Hi, On Mon, Feb 27, 2017 at 10:28 PM, Michael Niedermayer < mich...@niedermayer.cc> wrote: > Fixes: 686/clusterfuzz-testcase-5853946876788736 > > Found-by: continuous fuzzing process https://github.com/google/oss- > fuzz/tree/master/targets/ffmpeg > Signed-off-by: Michael Niedermayer

Re: [FFmpeg-devel] [PATCH v7] - Added Turing codec interface for ffmpeg

2017-02-28 Thread Mark Thompson
On 28/02/17 10:27, Saverio Blasi wrote: > Dear Mark, > > We provide our custom Boost libraries. In our tests we are able to build and > run using just "--enable-gpl --enable-libturing". We would prefer to keep the > pkgconfig file. I am not sure I fully understand what is required to be >

Re: [FFmpeg-devel] [PATCH] Remove qmin and qmax constraints for nvenc vbr

2017-02-28 Thread Miroslav Slugeň
Dne 28.2.2017 v 10:44 Timo Rothenpieler napsal(a): Am 28.02.2017 um 01:24 schrieb Ganapathy Raman Kasi: Can someone please help review patch. Thanks. I have seen an marked it, but I don't have time to review and test stuff right now. Will get to it eventually.

Re: [FFmpeg-devel] [PATCH v7] - Added Turing codec interface for ffmpeg

2017-02-28 Thread Saverio Blasi
Dear Mark, We provide our custom Boost libraries. In our tests we are able to build and run using just "--enable-gpl --enable-libturing". We would prefer to keep the pkgconfig file. I am not sure I fully understand what is required to be changed there (if anything), could you please be a bit

Re: [FFmpeg-devel] [PATCH] Remove qmin and qmax constraints for nvenc vbr

2017-02-28 Thread Timo Rothenpieler
Am 28.02.2017 um 01:24 schrieb Ganapathy Raman Kasi: > Can someone please help review patch. Thanks. I have seen an marked it, but I don't have time to review and test stuff right now. Will get to it eventually. ___ ffmpeg-devel mailing list