[FFmpeg-devel] [PATCH] vp9: set color range to MPEG for intraonly profile 0

2017-08-18 Thread James Zern
this is undocumented in the vp9 bitstream and decoding specification doc, but matches libvpx Signed-off-by: James Zern --- libavcodec/vp9.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavcodec/vp9.c b/libavcodec/vp9.c index 7d8aced8c8..94430db9a3

Re: [FFmpeg-devel] [PATCH] vp9: set color range to MPEG for intraonly profile 0

2017-08-18 Thread James Zern
On Fri, Aug 18, 2017 at 8:31 PM, James Zern wrote: > this is undocumented in the vp9 bitstream and decoding specification > doc, but matches libvpx > this came up in comparing decode results of ffvp9 and libvpx with source from a hardware encoder. > Signed-off-by: James Zern

[FFmpeg-devel] [PATCH] vp9: set color range to MPEG for intraonly profile 0

2017-08-18 Thread James Zern
this is undocumented in the vp9 bitstream and decoding specification doc, but matches libvpx Signed-off-by: James Zern --- libavcodec/vp9.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavcodec/vp9.c b/libavcodec/vp9.c index 7d8aced8c8..94430db9a3

Re: [FFmpeg-devel] [PATCH v3] libavformat/mov: fix multiple stsd handling of files with edit list, fix #6584

2017-08-18 Thread Jiejun Zhang
On Fri, Aug 18, 2017 at 8:16 AM, Michael Niedermayer wrote: > > seeking still fails with this and the sample from #6584 is that > intended or a seperate issue ? I didn't see any failure using "./ffmpeg -ss 00:00:10" (and also other time from 0s to 20s). How to reproduce a

Re: [FFmpeg-devel] [PATCH 3/3] lavf/tls: verify TLS connections by default whenever possible

2017-08-18 Thread Rodger Combs
> On Aug 16, 2017, at 06:29, wm4 wrote: > > On Wed, 16 Aug 2017 02:19:18 -0500 > Rodger Combs > wrote: > >> This makes a reasonable effort to set the default configuration to behave >> securely, while maintaining

Re: [FFmpeg-devel] [mov] Fix trampling of ctts during seeks when sidx support is enabled.

2017-08-18 Thread Dale Curtis
Anything else here? It'd be nice to get this landed soon if no one has any other comments. - dale On Thu, Aug 10, 2017 at 1:02 PM, Dale Curtis wrote: > On Tue, Aug 8, 2017 at 6:48 PM, Michael Niedermayer < > mich...@niedermayer.cc> wrote: > >> >> the fate test seems to

Re: [FFmpeg-devel] [PATCH] Fix signed integer overflows

2017-08-18 Thread Vitaly Buka
Not sure or it's going to be very hard for me. third_party/ffmpeg/LGPL_pristine/libavformat/aviobuf.c:225:16 Error was: mov.c:3961:23: runtime error: signed integer overflow: 9223372036854775807 + 8 cannot be represented in type 'long' On Fri, Aug 18, 2017 at 1:13 AM, Carl Eugen Hoyos

Re: [FFmpeg-devel] [PATCH] Fix signed integer overflows

2017-08-18 Thread Vitaly Buka
On Fri, Aug 18, 2017 at 1:11 AM, Tomas Härdin wrote: > On 2017-08-18 08:14, Vitaly Buka wrote: > >> Signed integer overflow is undefined behavior. >> Detected with clang and -fsanitize=signed-integer-overflow >> >> Signed-off-by: Vitaly Buka >> --- >>

[FFmpeg-devel] [PATCH]lavc/g726: Add a little-endian G.726 encoder

2017-08-18 Thread Carl Eugen Hoyos
Hi! Attached patch allows to encode samples that the binary decoder attached to ticket #6596 can decode, they have the same format as the ITU reference samples ("right-justified"). Please comment, Carl Eugen From 5e50b1489a45207890580e801d216a5abce06344 Mon Sep 17 00:00:00 2001 From: Carl Eugen

[FFmpeg-devel] [PATCH] lavf/bink: fix latest header and add all existing revisions

2017-08-18 Thread bananaman255
From: bnnm KB2 'i' found in Life is Strange (Xbox 360), rest verified against binkconv.exe Signed-off-by: bnnm --- libavformat/bink.c | 17 + 1 file changed, 13 insertions(+), 4 deletions(-) diff --git a/libavformat/bink.c

Re: [FFmpeg-devel] [PATCH]lavd/libdc1394: Avoid a null pointer dereference

2017-08-18 Thread Carl Eugen Hoyos
2017-08-18 10:02 GMT+02:00 Carl Eugen Hoyos : > Attached patch - that I cannot test - is meant to fix a bug reported > at launchpad: > https://bugs.launchpad.net/ubuntu/+source/ffmpeg/+bug/1710849 OK'ed by Josh de Kock with a change to prevent a leak and applied. Carl Eugen

[FFmpeg-devel] [PATCH]lavf/g726: Add a raw little-endian G.726 demuxer

2017-08-18 Thread Carl Eugen Hoyos
Hi! Attached patch allows to read files produced by the "ITU compatible binary G726 encoder" attached to ticket #6596. Please comment, Carl Eugen From a145fb0c3bade64d1aca40ea02a651f4e65ffb2f Mon Sep 17 00:00:00 2001 From: Carl Eugen Hoyos Date: Fri, 18 Aug 2017 19:10:56

Re: [FFmpeg-devel] [PATCH 2/2] avcodec/pixlet: Fixes: undefined shift in av_mod_uintp2()

2017-08-18 Thread Paul B Mahol
On 8/18/17, Michael Niedermayer wrote: > Fixes: runtime error: shift exponent 4294967289 is too large for 32-bit type > 'int' > Fixes: 3030/clusterfuzz-testcase-minimized-4649809254285312 > > Found-by: continuous fuzzing process >

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

2017-08-18 Thread Maksym Veremeyenko
On 14.08.2017 0:00, Marton Balint wrote: [...] +ret = av_new_packet(pkt, v->yres * v->line_stride_in_bytes); +if (ret < 0) +return ret; You are leaking the NDI video frame here on error. Maybe better to put NDIlib_recv_free_video into the parent function. done [...] +

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

2017-08-18 Thread Maksym Veremeyenko
On 17.08.2017 11:42, Ali KIZIL wrote: [...] As general infomation, NDI v3 SDK is released at: http://pages.newtek.com/NDI-Developers-SDK-Download-Link.html i found two issues with v3, so wait for final issues resolving. -- Maksym Veremeyenko ___

[FFmpeg-devel] [PATCH 2/2] avcodec/pixlet: Fixes: undefined shift in av_mod_uintp2()

2017-08-18 Thread Michael Niedermayer
Fixes: runtime error: shift exponent 4294967289 is too large for 32-bit type 'int' Fixes: 3030/clusterfuzz-testcase-minimized-4649809254285312 Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg Signed-off-by: Michael Niedermayer

[FFmpeg-devel] [PATCH 1/2] avcodec/dirac_dwt_template: Fix integer overflow in vertical_compose53iL0()

2017-08-18 Thread Michael Niedermayer
Fixes: runtime error: signed integer overflow: 2147483646 + 2 cannot be represented in type 'int' Fixes: 3013/clusterfuzz-testcase-minimized-4644084197097472 Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg Signed-off-by: Michael Niedermayer

Re: [FFmpeg-devel] [PATCH] [PATCH] avdevice/avfoundation: add scaleFactor attribute for avfoundation

2017-08-18 Thread sharp bai
Thank you for your patient. I will follow your direction. Question 1: Your patch misses to cover the scale factor in case of a device name string given by the user. The problem in mac os the screen to be catpured in avfoundation not have a real name. Only video device in avfoundation has a name.

Re: [FFmpeg-devel] [PATCH 1/1] libavdevice/decklink: configurablity to set max queue size

2017-08-18 Thread Matthias Hunstock
Am 14.08.2017 um 06:55 schrieb Patagar, Ravindra: > + * Copyright (c) 2017 Akamai Technologies, Inc. Isn't this kind of exaggerated for this little patch? Or is it company policy to do it? Beside that I'd prefer the person over the employer or at least both. Matthias

Re: [FFmpeg-devel] [PATCH] vdpau: do not use buggy HEVC support by default

2017-08-18 Thread wm4
On Tue, 8 Aug 2017 05:15:52 + Manoj Bonda wrote: > Hi , > > HEVC issue for read-back API has been fixed and will be part of the upcoming > drivers. > Please help us understand the issue with the open gl interop. That's great to hear. (Sorry, saw this only now...) >

Re: [FFmpeg-devel] [PATCH 1/1] libavdevice/decklink: configurablity to set max queue size

2017-08-18 Thread Patagar, Ravindra
Hi Marton, Thanks for the review. Please find the updated patch attached. Regards, Ravindra On 8/18/17, 1:09 AM, "Marton Balint" wrote: On Thu, 17 Aug 2017, Patagar, Ravindra wrote: > Hi Marton, > > I have updated the patch as per your comments.

Re: [FFmpeg-devel] [PATCH 1/2] avcodec/pixlet: fixes integer overflow in read_highpass()

2017-08-18 Thread Michael Niedermayer
On Thu, Aug 17, 2017 at 03:54:56AM +0200, Michael Niedermayer wrote: > Fixes: runtime error: negation of -2147483648 cannot be represented in type > 'int32_t' (aka 'int'); cast to an unsigned type to negate this value to itself > Fixes: 2879/clusterfuzz-testcase-minimized-6317542639403008 > >

Re: [FFmpeg-devel] [PATCH 1/2] avcodec/rangecoder: Do not increase the pointer beyond the buffer

2017-08-18 Thread Michael Niedermayer
On Wed, Aug 16, 2017 at 06:27:19PM +0200, Michael Niedermayer wrote: > On Mon, Aug 14, 2017 at 11:30:06PM -0300, James Almer wrote: > > On 8/14/2017 8:07 PM, Michael Niedermayer wrote: > > > On Sun, Aug 13, 2017 at 07:18:11PM -0300, James Almer wrote: > > >> On 8/13/2017 7:15 PM, Michael

Re: [FFmpeg-devel] [PATCH] Fix signed integer overflows

2017-08-18 Thread Carl Eugen Hoyos
2017-08-18 8:14 GMT+02:00 Vitaly Buka : > Signed integer overflow is undefined behavior. > Detected with clang and -fsanitize=signed-integer-overflow > --- a/libavformat/mov.c > +++ b/libavformat/mov.c > @@ -5572,7 +5572,7 @@ static int

Re: [FFmpeg-devel] [PATCH] Fix signed integer overflows

2017-08-18 Thread Tomas Härdin
On 2017-08-18 08:14, Vitaly Buka wrote: Signed integer overflow is undefined behavior. Detected with clang and -fsanitize=signed-integer-overflow Signed-off-by: Vitaly Buka --- libavcodec/utils.c| 2 +- libavformat/aviobuf.c | 4 +++- libavformat/mov.c | 2 +-

[FFmpeg-devel] [PATCH]lavd/libdc1394: Avoid a null pointer dereference

2017-08-18 Thread Carl Eugen Hoyos
Hi! Attached patch - that I cannot test - is meant to fix a bug reported at launchpad: https://bugs.launchpad.net/ubuntu/+source/ffmpeg/+bug/1710849 Please comment, Carl Eugen From 6db6b36a36da95de9b2bae34313710df5b126811 Mon Sep 17 00:00:00 2001 From: Carl Eugen Hoyos Date:

[FFmpeg-devel] [PATCH] Fix signed integer overflows

2017-08-18 Thread Vitaly Buka
Signed integer overflow is undefined behavior. Detected with clang and -fsanitize=signed-integer-overflow Signed-off-by: Vitaly Buka --- libavcodec/utils.c| 2 +- libavformat/aviobuf.c | 4 +++- libavformat/mov.c | 2 +- 3 files changed, 5 insertions(+), 3