[FFmpeg-devel] [PATCH] Implement muxing AAC in a CAFF container

2018-11-09 Thread Lewis Fox
From: Lewis Fox Signed-off-by: Lewis Fox --- libavformat/cafenc.c | 61 -- libavformat/mov.c | 1 + libavformat/mov_esds.c | 1 - 3 files changed, 54 insertions(+), 9 deletions(-) diff --git a/libavformat/cafenc.c b/libavformat/cafenc.c index

[FFmpeg-devel] Implement muxing AAC in a CAFF container

2018-11-09 Thread Lewis Fox
I was messing around with the CAFF support, and noticed that muxing AAC had been noted as unsupported for a long time. I looked into it, and realized that by using code from movenc.c, this could be fairly trivially added. I tested this implementation by playing the resulting CAFF files in

[FFmpeg-devel] Implement muxing AAC in a CAFF container

2018-11-09 Thread Lewis Fox
I was messing around with the CAFF support, and noticed that muxing AAC had been noted as unsupported for a long time. I looked into it, and realized that by using code from movenc.c, this could be fairly trivially added. I tested this implementation by playing the resulting CAFF files in

[FFmpeg-devel] Implement muxing AAC in a CAFF container

2018-11-09 Thread Lewis Fox
I was messing around with the CAFF support, and noticed that muxing AAC had been noted as unsupported for a long time. I looked into it, and realized that by using code from movenc.c, this could be fairly trivially added. I tested this implementation by playing the resulting CAFF files in

Re: [FFmpeg-devel] [PATCH] libvpxenc,vp9: add enable-tpl option

2018-11-09 Thread James Zern
On Thu, Nov 8, 2018 at 2:46 PM James Zern wrote: > > On Sat, Nov 3, 2018 at 2:01 PM James Zern wrote: > > > > enables temporal dependency model > > > > Signed-off-by: James Zern > > --- > > doc/encoders.texi | 2 ++ > > libavcodec/libvpxenc.c | 11 +++ > > 2 files changed, 13

Re: [FFmpeg-devel] [PATCH 1/3] avcodec/imm4: Use ff_set_dimensions()

2018-11-09 Thread Paul B Mahol
On 11/9/18, Michael Niedermayer wrote: > Fixes: Out of memory > Fixes: > 10970/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_IMM4_fuzzer-5698750043914240 > > Found-by: continuous fuzzing process > https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg > Signed-off-by: Michael

[FFmpeg-devel] [PATCH 2/3] avcodec/shorten: Fix integer overflow with offset

2018-11-09 Thread Michael Niedermayer
Fixes: signed integer overflow: -1625810908 - 582229060 cannot be represented in type 'int' Fixes: 10977/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_SHORTEN_fuzzer-5732602018267136 Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg

[FFmpeg-devel] [PATCH 1/3] avcodec/imm4: Use ff_set_dimensions()

2018-11-09 Thread Michael Niedermayer
Fixes: Out of memory Fixes: 10970/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_IMM4_fuzzer-5698750043914240 Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg Signed-off-by: Michael Niedermayer --- libavcodec/imm4.c | 5 +++-- 1 file

[FFmpeg-devel] [PATCH 3/3] avcodec/golomb: Speed up long ur_golomb codes

2018-11-09 Thread Michael Niedermayer
Fixes: Timeout Fixes: 10972/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_FLAC_fuzzer-5707569640243200 Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg Signed-off-by: Michael Niedermayer --- libavcodec/golomb.h | 10 +++--- 1 file

Re: [FFmpeg-devel] [PATCH] avcodec/pngdec: Check compression method

2018-11-09 Thread Paul B Mahol
On 11/9/18, Carl Eugen Hoyos wrote: > 2018-11-09 10:31 GMT+01:00, Michael Niedermayer : >> method 0 (inflate/deflate) is the only specified in the specification and >> the only supported >> >> Fixes: Timeout >> Fixes: >>

Re: [FFmpeg-devel] [PATCH] avcodec/pngdec: Check compression method

2018-11-09 Thread Carl Eugen Hoyos
2018-11-09 10:31 GMT+01:00, Michael Niedermayer : > method 0 (inflate/deflate) is the only specified in the specification and > the only supported > > Fixes: Timeout > Fixes: > 10976/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_PNG_fuzzer-5729372588736512 > > Found-by: continuous fuzzing

Re: [FFmpeg-devel] [PATCH V2 2/3] fftools/ffmpeg: Put the variable declaration at uppper for block.

2018-11-09 Thread Carl Eugen Hoyos
2018-11-09 14:11 GMT+01:00, Jun Zhao : > move the variable declaration at start of upper for block and > remove the redundant brace. Why? (You never had to use git blame and were unhappy about the necessary iterations?) No objections here, Carl Eugen

[FFmpeg-devel] [PATCH V2 2/3] fftools/ffmpeg: Put the variable declaration at uppper for block.

2018-11-09 Thread Jun Zhao
move the variable declaration at start of upper for block and remove the redundant brace. Signed-off-by: Jun Zhao --- fftools/ffmpeg.c |7 ++- 1 files changed, 2 insertions(+), 5 deletions(-) diff --git a/fftools/ffmpeg.c b/fftools/ffmpeg.c index 9dc42f5..68b8e05 100644 ---

[FFmpeg-devel] [PATCH V2 1/3] fftools/ffmpeg: Remove the micor like "#if 1"

2018-11-09 Thread Jun Zhao
They are come from 2003 and delete them. Signed-off-by: Jun Zhao --- fftools/ffmpeg.c |6 -- 1 files changed, 0 insertions(+), 6 deletions(-) diff --git a/fftools/ffmpeg.c b/fftools/ffmpeg.c index da4259a..9dc42f5 100644 --- a/fftools/ffmpeg.c +++ b/fftools/ffmpeg.c @@ -1210,11 +1210,7

[FFmpeg-devel] [PATCH V2 3/3] fftools/ffmpeg: Indent the code

2018-11-09 Thread Jun Zhao
Signed-off-by: Jun Zhao --- fftools/ffmpeg.c | 128 +++--- 1 files changed, 64 insertions(+), 64 deletions(-) diff --git a/fftools/ffmpeg.c b/fftools/ffmpeg.c index 68b8e05..38c21e9 100644 --- a/fftools/ffmpeg.c +++ b/fftools/ffmpeg.c @@ -1193,27

[FFmpeg-devel] [PATCH V2 0/3] Misc change in ffmpeg.c

2018-11-09 Thread Jun Zhao
V2: - split the patch as Carl's comment. - more details in commit message as Micheal's comment. Jun Zhao (3): fftools/ffmpeg: Remove the micor like "#if 1" fftools/ffmpeg: Put the variable declaration at uppper for block. fftools/ffmpeg: Indent the code fftools/ffmpeg.c | 137

Re: [FFmpeg-devel] [PATCH V4] Add a filter implementing HDR image generation from a single exposure using deep CNNs

2018-11-09 Thread Guo, Yejun
this filter accepts 8bit frame (RGB24) and outputs 10bit/float frame, and there's no reference image, so it is not feasible to use criteria such as PNSR, SSIM. I choose the same method described in the paper to demo the filter effect, that means the frames before/after the filter are reduced

Re: [FFmpeg-devel] [PATCH v3] fftools/ffmpeg: add an option to forbid the fallback to software path when hardware init fails

2018-11-09 Thread Li, Zhong
> From: ffmpeg-devel [mailto:ffmpeg-devel-boun...@ffmpeg.org] On Behalf > Of Linjie Fu > Sent: Friday, November 9, 2018 5:06 PM > To: ffmpeg-devel@ffmpeg.org > Cc: Fu, Linjie > Subject: [FFmpeg-devel] [PATCH v3] fftools/ffmpeg: add an option to forbid > the fallback to software path when hardware

[FFmpeg-devel] [PATCH] avcodec/pngdec: Check compression method

2018-11-09 Thread Michael Niedermayer
method 0 (inflate/deflate) is the only specified in the specification and the only supported Fixes: Timeout Fixes: 10976/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_PNG_fuzzer-5729372588736512 Found-by: continuous fuzzing process

Re: [FFmpeg-devel] [PATCH 2/4] lavfi/opencl: Handle overlay input formats correctly.

2018-11-09 Thread Li, Zhong
> > > > > > > > The main input may have alpha channel, we just ignore it. > > > > > > > > Also add some checks for incompatible input formats. > > > > > > > > > > > > > > > > Signed-off-by: Ruiling Song > > > > > > LGTM. > > > > > > BTW, could the main input with alpha case be supported? > > > >

Re: [FFmpeg-devel] avfilter/af_headphone : fix memleak

2018-11-09 Thread Paul B Mahol
On 11/8/18, Martin Vignali wrote: > Hello, > > Patch in attach should fix the memleak report by coverity > CID 1439934 > CID 1439935 > > Replace return ret, by goto fail. > > Martin > ok ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org

Re: [FFmpeg-devel] [PATCH] libavformat/ffmetadec: use dynamic allocation for line buffer

2018-11-09 Thread Marton Balint
On Thu, 8 Nov 2018, François Revol wrote: Hi, Le 08/11/2018 à 20:50, Michael Niedermayer a écrit : +line = av_malloc(line_size); +if (!line) +return AVERROR(ENOMEM); this would use alot of memory for large files, also avio_size() will not work with all inputs Yes I

[FFmpeg-devel] [PATCH v3] fftools/ffmpeg: add an option to forbid the fallback to software path when hardware init fails

2018-11-09 Thread Linjie Fu
Currently ff_get_format will go through all usable choices if the chosen format was not supported. It will fallback to software path if the hardware init fails. Provided an option "-require_hwaccel 1" in user-code to detect frame->format and hwaccel_get_buffer in get_buffer. If hardware init

Re: [FFmpeg-devel] [PATCH v2] fftools/ffmpeg: add an option to forbid the fallback to software path when hardware init fails

2018-11-09 Thread Fu, Linjie
> -Original Message- > From: Eoff, Ullysses A > Sent: Friday, November 9, 2018 11:34 > To: FFmpeg development discussions and patches de...@ffmpeg.org> > Cc: Fu, Linjie > Subject: RE: [FFmpeg-devel] [PATCH v2] fftools/ffmpeg: add an option to > forbid the fallback to software path when

Re: [FFmpeg-devel] [PATCH 2/4] lavfi/opencl: Handle overlay input formats correctly.

2018-11-09 Thread Song, Ruiling
> -Original Message- > From: ffmpeg-devel [mailto:ffmpeg-devel-boun...@ffmpeg.org] On Behalf Of > Li, Zhong > Sent: Thursday, November 8, 2018 11:39 AM > To: FFmpeg development discussions and patches > Subject: Re: [FFmpeg-devel] [PATCH 2/4] lavfi/opencl: Handle overlay input > formats