Re: [FFmpeg-devel] [PATCH 1/2] error_resilience: remove avpriv_atomic usage

2017-11-24 Thread James Almer
On 11/24/2017 9:33 PM, Rostislav Pehlivanov wrote: > Signed-off-by: Rostislav Pehlivanov > --- > libavcodec/error_resilience.c | 12 ++-- > libavcodec/error_resilience.h | 3 ++- > 2 files changed, 8 insertions(+), 7 deletions(-) > > diff --git a/libavcodec/error_resilience.c b/libavcod

Re: [FFmpeg-devel] [PATCH] ffmpeg libopusdec: fix missing include file in libopusdec.c

2017-11-24 Thread Michael Niedermayer
On Fri, Nov 24, 2017 at 12:30:55AM +0100, Carl Eugen Hoyos wrote: > 2017-11-23 23:41 GMT+01:00 Hendrik Leppkes : > > On Thu, Nov 23, 2017 at 8:22 PM, Mikulas Patocka > > wrote: > >> This patch fixes the following error when compiling mplayer with libopus. > >> > >> libavcodec/libopusdec.c: In fun

Re: [FFmpeg-devel] [PATCH 2/2] avcodec/sbrdsp_fixed: Fix integer overflow

2017-11-24 Thread Michael Niedermayer
On Thu, Nov 23, 2017 at 11:00:21PM +0100, Michael Niedermayer wrote: > On Wed, Nov 22, 2017 at 11:59:30PM +0100, Hendrik Leppkes wrote: > > On Wed, Nov 22, 2017 at 11:38 PM, Carl Eugen Hoyos > > wrote: > > > 2017-11-22 21:00 GMT+01:00 Michael Niedermayer : > > > > > >> diff --git a/libavcodec/sbr

[FFmpeg-devel] [PATCH] configure: remove superfluous cuvid and nvdec checks

2017-11-24 Thread James Almer
Both are autodetected, and their dependency on cuda is checked elsewhere. Fixes ticket #6849. --- configure | 4 1 file changed, 4 deletions(-) diff --git a/configure b/configure index 3ec6407fb2..7769427ffb 100755 --- a/configure +++ b/configure @@ -5816,10 +5816,6 @@ done # these are o

Re: [FFmpeg-devel] [PATCH] tests/checkasm/float_dsp: Increase allowed difference for float_dsp.vector_dmul

2017-11-24 Thread James Almer
On 10/29/2017 11:57 AM, Michael Niedermayer wrote: > The choosen value is the lowest power of 2 that allows 1000 iterations of > fate-checkasm-float_dsp > to pass on x86-32 Ticket #6848 reports this value is still not enough. Maybe something like 1.0e-12 or 1.0e-13 instead? > > Signed-off-by: M

[FFmpeg-devel] [PATCH] avcodec/dirac_dwt: Fix integer overflow in COMPOSE_FIDELITYi*

2017-11-24 Thread Michael Niedermayer
Fixes: runtime error: signed integer overflow: -2143827186 - 7404944 cannot be represented in type 'int' Fixes: 4354/clusterfuzz-testcase-minimized-4671122764201984 Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg Signed-off-by: Michael Niederma

Re: [FFmpeg-devel] [PATCH 1/2] error_resilience: remove avpriv_atomic usage

2017-11-24 Thread Michael Niedermayer
On Fri, Nov 24, 2017 at 10:18:06PM -0300, James Almer wrote: > On 11/24/2017 9:33 PM, Rostislav Pehlivanov wrote: > > Signed-off-by: Rostislav Pehlivanov > > --- > > libavcodec/error_resilience.c | 12 ++-- > > libavcodec/error_resilience.h | 3 ++- > > 2 files changed, 8 insertions(+),

Re: [FFmpeg-devel] [PATCH 1/2] lavf/flacenc: support writing attached pictures

2017-11-24 Thread Michael Niedermayer
On Fri, Nov 24, 2017 at 08:53:55PM -0300, James Almer wrote: > On 11/24/2017 8:27 PM, Michael Niedermayer wrote: > > On Thu, Nov 23, 2017 at 07:08:42PM -0300, James Almer wrote: > >> From: Rodger Combs > >> > >> Signed-off-by: James Almer > >> --- > >> Should be good to commit now. > >> > >> lib

[FFmpeg-devel] [PATCH v5 1/2] avformat/hlsenc: Modularized playlist creation to allow reuse

2017-11-24 Thread Karthick J
--- libavformat/Makefile | 2 +- libavformat/hlsenc.c | 115 +++--- libavformat/hlsplaylist.c | 138 ++ libavformat/hlsplaylist.h | 51 + 4 files changed, 211 insertions(+), 95 deletions(-) cr

Re: [FFmpeg-devel] [PATCH v4 1/2] avformat/hlsenc: Modularized playlist creation to allow reuse

2017-11-24 Thread Jeyapal, Karthick
Oops. Sorry about that. Have sent a new patch v5 that fixes this. Regards, Karthick On 11/25/17, 1:31 AM, "Michael Niedermayer" wrote: On Fri, Nov 24, 2017 at 10:57:45AM +0530, Karthick J wrote: > --- > libavformat/Makefile | 2 +- > libavformat/hlsenc.c | 115 +++--

[FFmpeg-devel] [PATCH] web/index: add news entry for release 3.4

2017-11-24 Thread James Almer
Signed-off-by: James Almer --- This was for some reason missed back in October. src/index | 52 1 file changed, 52 insertions(+) diff --git a/src/index b/src/index index 0ca7a10..2b44762 100644 --- a/src/index +++ b/src/index @@ -37,6 +37,58

Re: [FFmpeg-devel] [PATCH 0/1][TOOL][HACK] Allocation NULL check fuzzer

2017-11-24 Thread Vittorio Giovara
On 11/24/2017 11:35 PM, Derek *Buitenhuis* wrote: > It would probably make an absolute ton of reports, since there are quite > a few unchecked allocs in FFmpeg... might be kinda spammy. That might not be the case any more, most of the checks have been added in the Coverity effort of 2015 and cont

<    1   2