Re: [FFmpeg-devel] [PATCH] doc/libav-merge: remove the hls merge TODO

2017-07-26 Thread Clément Bœsch
On Thu, Jul 27, 2017 at 11:45:32AM +0800, Steven Liu wrote: > 2017-07-25 11:42 GMT+08:00 Steven Liu : > > From: Steven Liu > > > > because the commit id 3c7fa8cbb93dba96acb12aef6e00d929c1da1ae8 > > have been done for OPENSSL at hlsenc. > > I don't

Re: [FFmpeg-devel] [PATCH] doc/libav-merge: remove the hls merge TODO

2017-07-26 Thread Steven Liu
2017-07-25 11:42 GMT+08:00 Steven Liu : > From: Steven Liu > > because the commit id 3c7fa8cbb93dba96acb12aef6e00d929c1da1ae8 > have been done for OPENSSL at hlsenc. > > Signed-off-by: Steven Liu > --- > doc/libav-merge.txt | 1 - > 1

[FFmpeg-devel] How to parse the coeff_token of H264 cavlc

2017-07-26 Thread He Lei
The describe of coeff_token is ce(v) in bitstream. How do I to parse it`s value from bitstream ? Have any guys know? Orz... ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH] avcodec/dnxhddec: Move mb height check out of non hr branch

2017-07-26 Thread Michael Niedermayer
On Wed, Jul 26, 2017 at 03:26:59AM +0200, Michael Niedermayer wrote: > Fixes: out of array access > Fixes: poc.dnxhd > > Found-by: Bingchang, Liu@VARAS of IIE > Signed-off-by: Michael Niedermayer > --- > libavcodec/dnxhddec.c | 8 ++-- > 1 file changed, 6

Re: [FFmpeg-devel] [PATCH]v6 Opus Pyramid Vector Quantization Search in x86 SIMD asm

2017-07-26 Thread Rostislav Pehlivanov
On 26 July 2017 at 15:56, Ivan Kalvachev wrote: > +if (ARCH_X86 && CONFIG_OPUS_ENCODER) > +ff_opus_dsp_init_x86(s); > Just change it to +if (ARCH_X86) The init function is named opus_dsp, so it'll get used to other opus things, not just the encoder. The

[FFmpeg-devel] [PATCH] libavformat/mov: Fix inserting frames before current_frame

2017-07-26 Thread Jacob Trimble
When using streaming input, it may be possible to see frames that appear before the current_frame. When these frames are inserted into the index, the current_frame needs to be updated so it is still pointing at the same frame. Signed-off-by: Jacob Trimble ---

[FFmpeg-devel] [PATCH] avcodec/pixlet: Simplify nbits computation

2017-07-26 Thread Michael Niedermayer
Fixes multiple integer overflows Fixes: runtime error: signed integer overflow: 1 + 2147483647 cannot be represented in type 'int' Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg Signed-off-by: Michael Niedermayer ---

Re: [FFmpeg-devel] [PATCH 3/3] avformat/s337m: fix potentially undefined pointer arithmetic

2017-07-26 Thread foo86
On Mon, Jul 24, 2017 at 03:27:06PM +0300, foo86 wrote: > > Use integer position instead of pointer for loop variable. Also only > skip header fields after header has been fully validated. > --- > libavformat/s337m.c | 20 ++-- > 1 file changed, 10 insertions(+), 10 deletions(-)

[FFmpeg-devel] [PATCH] avcodec/diracdec: Fix integer overflow in signed multiplication in UNPACK_ARITH()

2017-07-26 Thread Michael Niedermayer
Fixes: runtime error: signed integer overflow: 1073741823 * 4 cannot be represented in type 'int' Fixes: 2729/clusterfuzz-testcase-minimized-5902915464069120 Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg Signed-off-by: Michael Niedermayer

[FFmpeg-devel] [PATCH] avcodec/aacdec: Fix PCE channel_layout verification

2017-07-26 Thread nsugino
From: Nicolas Sugino --- libavcodec/aacdec_template.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavcodec/aacdec_template.c b/libavcodec/aacdec_template.c index 90cc143..a539f74 100644 --- a/libavcodec/aacdec_template.c +++

Re: [FFmpeg-devel] [PATCH] fate: update pixfmt_best test to check for endianness

2017-07-26 Thread Michael Niedermayer
On Wed, Jul 26, 2017 at 03:58:54PM +0200, Tobias Rapp wrote: > Signed-off-by: Tobias Rapp > --- > libavutil/tests/pixfmt_best.c | 14 ++ > tests/ref/fate/pixfmt_best| 2 +- > 2 files changed, 15 insertions(+), 1 deletion(-) LGTM thx [...] -- Michael

[FFmpeg-devel] [PATCH]v6 Opus Pyramid Vector Quantization Search in x86 SIMD asm

2017-07-26 Thread Ivan Kalvachev
On 7/24/17, Ivan Kalvachev wrote: > On 7/23/17, Rostislav Pehlivanov wrote: >> On 22 July 2017 at 12:18, Ivan Kalvachev wrote: >> >>> This patch is ready for review and inclusion. >>> >>> >>> >>>+%macro emu_blendvps 3 ; dst/src_a,

Re: [FFmpeg-devel] [PATCH V2 1/2] avcodec/vorbisenc: Add pre-echo detection

2017-07-26 Thread Tyler Jones
On Wed, Jul 26, 2017 at 12:51:31AM +0100, Rostislav Pehlivanov wrote: > On 17 July 2017 at 16:17, Tyler Jones wrote: > > > +float last_var; > > +const float eps = 1e-4; > > > > Use normal notation for floats and add an f at the end to inform the > compiler the

Re: [FFmpeg-devel] [PATCH 2/3] ffprobe: add support for ICC Profile frame side data

2017-07-26 Thread James Almer
On 7/26/2017 3:10 AM, Rostislav Pehlivanov wrote: > On 26 July 2017 at 05:31, James Almer wrote: > >> Print the name metadata entry and the buffer size. >> >> Signed-off-by: James Almer >> --- >> ffprobe.c | 5 + >> 1 file changed, 5 insertions(+) >>

Re: [FFmpeg-devel] [PATCH 3/3] avcodec/webp: add support for ICCP chunks

2017-07-26 Thread James Almer
On 7/26/2017 3:18 AM, Rostislav Pehlivanov wrote: > On 26 July 2017 at 05:31, James Almer wrote: > >> Export the raw data as ICC Profile frame side data. >> >> Signed-off-by: James Almer >> --- >> libavcodec/webp.c | 26 -- >> 1

Re: [FFmpeg-devel] [PATCH 1/3] avutil/frame: add ICC Profile to av_frame_side_data_name

2017-07-26 Thread James Almer
On 7/26/2017 3:06 AM, Rostislav Pehlivanov wrote: > On 26 July 2017 at 05:31, James Almer wrote: > >> Signed-off-by: James Almer >> --- >> libavutil/frame.c | 1 + >> 1 file changed, 1 insertion(+) >> >> diff --git a/libavutil/frame.c b/libavutil/frame.c

Re: [FFmpeg-devel] [PATCH 4/4] libavcodec/dev: v4l2: add support for v4l2 mem2mem codecs

2017-07-26 Thread Michael Niedermayer
On Mon, Jul 24, 2017 at 09:08:49PM +0200, Jorge Ramirez-Ortiz wrote: > From: Alexis Ballier > > This patchset enhances Alexis Ballier's original patch and validates > it using Qualcomm's Venus hardware (driver recently landed upstream > [1]). > > This has been tested on

[FFmpeg-devel] [PATCH] fate: update pixfmt_best test to check for endianness

2017-07-26 Thread Tobias Rapp
Signed-off-by: Tobias Rapp --- libavutil/tests/pixfmt_best.c | 14 ++ tests/ref/fate/pixfmt_best| 2 +- 2 files changed, 15 insertions(+), 1 deletion(-) diff --git a/libavutil/tests/pixfmt_best.c b/libavutil/tests/pixfmt_best.c index a617633..e98fcc1

Re: [FFmpeg-devel] [PATCH 2/2] avcodec/jpeg2000dec: Fix division by zero in jp2_find_codestream()

2017-07-26 Thread Michael Niedermayer
On Mon, Jul 24, 2017 at 07:56:34PM +0200, Carl Eugen Hoyos wrote: > 2017-07-24 17:34 GMT+02:00 Michael Niedermayer : > > Fixes: 2707/clusterfuzz-testcase-minimized-5179636394754048 > > > > Found-by: continuous fuzzing process > >

Re: [FFmpeg-devel] [PATCH V2 1/2] avcodec/vorbisenc: Add pre-echo detection

2017-07-26 Thread Moritz Barsnick
On Wed, Jul 26, 2017 at 00:51:31 +0100, Rostislav Pehlivanov wrote: > > +float last_var; > > +const float eps = 1e-4; > > Use normal notation for floats and add an f at the end to inform the > compiler the constant is a float. Since I've been nitpicking float/double promotion issues

Re: [FFmpeg-devel] [PATCH 3/3] avcodec/webp: add support for ICCP chunks

2017-07-26 Thread Moritz Barsnick
On Wed, Jul 26, 2017 at 07:18:35 +0100, Rostislav Pehlivanov wrote: > > * Exif metadata > > + * ICC profile > > Can't find anything that could go wrong here and the specs don't mention > anything special, so LGTM. Does this perhaps imply need of a micro version bump? Moritz

Re: [FFmpeg-devel] [Patch] Auto Insert of hwupload_cuda filter

2017-07-26 Thread Nicolas George
Le septidi 7 thermidor, an CCXXV, Yogender Gupta a écrit : > Currently combining CPU and CUDA filters requires insertion of hwupload and > download filters. I am trying to simply this by auto insertion of these > filters. This patch is for hwupload_cuda, but I want to do this for > hwdownload

Re: [FFmpeg-devel] [Patch] nvenc : Add P016 support

2017-07-26 Thread Philip Langdale
On 2017-07-25 08:33, Yogender Gupta wrote: I can push, but to answer Timo's question: I did not add P016 to nvenc because it is technically not correct to pass 12bit content through a 10bit surface - that will lead to truncation, rather than dithering. If you are telling us that nvenc supports

Re: [FFmpeg-devel] [Patch] Auto Insert of hwupload_cuda filter

2017-07-26 Thread Philip Langdale
On 2017-07-25 09:41, Yogender Gupta wrote: Currently combining CPU and CUDA filters requires insertion of hwupload and download filters. I am trying to simply this by auto insertion of these filters. This patch is for hwupload_cuda, but I want to do this for hwdownload as well. The attached

Re: [FFmpeg-devel] [PATCH] Implement NewTek NDI support

2017-07-26 Thread Maksym Veremeyenko
On 24.07.2017 20:11, Nicolas George wrote: Le quintidi 5 thermidor, an CCXXV, Maksym Veremeyenko a écrit : would it be ok: Subject: [PATCH] lavf: implement NewTek NDI support Exactly. Except I mistyped: that would be lavd, not lavf; my bad. fixed would you *newtek_ndi* or

Re: [FFmpeg-devel] [PATCH 3/3] avcodec/webp: add support for ICCP chunks

2017-07-26 Thread Rostislav Pehlivanov
On 26 July 2017 at 05:31, James Almer wrote: > Export the raw data as ICC Profile frame side data. > > Signed-off-by: James Almer > --- > libavcodec/webp.c | 26 -- > 1 file changed, 24 insertions(+), 2 deletions(-) > > diff --git

Re: [FFmpeg-devel] [PATCH 2/3] ffprobe: add support for ICC Profile frame side data

2017-07-26 Thread Rostislav Pehlivanov
On 26 July 2017 at 05:31, James Almer wrote: > Print the name metadata entry and the buffer size. > > Signed-off-by: James Almer > --- > ffprobe.c | 5 + > 1 file changed, 5 insertions(+) > > diff --git a/ffprobe.c b/ffprobe.c > index

Re: [FFmpeg-devel] [PATCH 1/3] avutil/frame: add ICC Profile to av_frame_side_data_name

2017-07-26 Thread Rostislav Pehlivanov
On 26 July 2017 at 05:31, James Almer wrote: > Signed-off-by: James Almer > --- > libavutil/frame.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/libavutil/frame.c b/libavutil/frame.c > index 24d5d5f184..f01b4d7b64 100644 > ---