Re: [FFmpeg-devel] [PATCH] avcodec/scpr: mask bits to prevent out of array read

2017-05-15 Thread Paul B Mahol
On 5/16/17, Michael Niedermayer wrote: > Fixes: 1615/clusterfuzz-testcase-minimized-6625214647500800 > > Found-by: continuous fuzzing process > https://github.com/google/oss-fuzz/tree/master/targets/ffmpeg > Signed-off-by: Michael Niedermayer > --- > libavcodec/scpr.c | 4 ++-- > 1 file changed,

Re: [FFmpeg-devel] [PATCH] libavformat aviobuf: Fixed dst pointer initialization in fill_buffer

2017-05-15 Thread wm4
On Mon, 15 May 2017 17:55:28 + Rob Meyers wrote: > Of course. > > We noticed when reading data from a named pipe the first 10 bytes would get > dropped. I traced this to the affected code in fill_buffer(). The > assignment of "dst" was always set to the beginning of the buffer, and if > it h

Re: [FFmpeg-devel] [PATCH] movenc/isom: update vpcC box to the latest draft specification

2017-05-15 Thread Hendrik Leppkes
On Tue, Apr 25, 2017 at 11:11 AM, Hendrik Leppkes wrote: > On Mon, Apr 24, 2017 at 11:59 PM, James Almer wrote: >> On 4/24/2017 6:48 PM, KongQun Yang wrote: >>> -- KongQun Yang (KQ) >>> >>> On Fri, Apr 21, 2017 at 4:49 PM, Hendrik Leppkes >>> wrote: >>> On Sat, Apr 22, 2017 at 1:25 AM, Hend

[FFmpeg-devel] [PATCH] avcodec/scpr: mask bits to prevent out of array read

2017-05-15 Thread Michael Niedermayer
Fixes: 1615/clusterfuzz-testcase-minimized-6625214647500800 Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/targets/ffmpeg Signed-off-by: Michael Niedermayer --- libavcodec/scpr.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/libavco

Re: [FFmpeg-devel] [PATCH] build: remove --enable-raise-major configure option

2017-05-15 Thread James Almer
On 5/6/2017 6:59 PM, James Almer wrote: > It's not used by anything, has dubious usefulness, the reasons for which > it was introduced are no longer valid, and only serves to add complexity > to the build system. > > Signed-off-by: James Almer > --- > Makefile | 6 -- > configur

[FFmpeg-devel] [PATCH] Added line to the af_bs2b filter docs mentioning --enable-libbs2b

2017-05-15 Thread Leo Izen
--- doc/filters.texi | 3 +++ 1 file changed, 3 insertions(+) diff --git a/doc/filters.texi b/doc/filters.texi index 9611a41e28..a7c6063969 100644 --- a/doc/filters.texi +++ b/doc/filters.texi @@ -1911,6 +1911,9 @@ available are filtered. Bauer stereo to binaural transformation, which improves h

Re: [FFmpeg-devel] [PATCH] hevc: Fix scaling list prediction delta for the 32x32 inter matrix

2017-05-15 Thread Mark Thompson
On 15/05/17 22:46, James Almer wrote: > On 5/15/2017 5:29 PM, Mark Thompson wrote: >> On 30/04/17 13:34, Mark Thompson wrote: >>> Fixes ticket #6356. >>> --- >>> This was broken in 627c044f50da3046809314f7cc742b8a10cf26a1, which changed >>> the array index of the 32x32 inter matrix but missed this

Re: [FFmpeg-devel] [PATCH] hevc: Fix scaling list prediction delta for the 32x32 inter matrix

2017-05-15 Thread James Almer
On 5/15/2017 5:29 PM, Mark Thompson wrote: > On 30/04/17 13:34, Mark Thompson wrote: >> Fixes ticket #6356. >> --- >> This was broken in 627c044f50da3046809314f7cc742b8a10cf26a1, which changed >> the array index of the 32x32 inter matrix but missed this case. >> >> Sample at

Re: [FFmpeg-devel] FFMpeg GPL license Violations by Android apps

2017-05-15 Thread riya khanna
AFAIK, all Android apps are executed under Java runtime virtual machine sandbox (they cannot fork and start a new process). All shared libraries shipped with app installation packages (e..g, libffmpeg.so) are loaded dynamically from Java runtime using the JNI interface into the host process address

Re: [FFmpeg-devel] FFMpeg GPL license Violations by Android apps

2017-05-15 Thread Aaron Levinson
On 5/15/2017 1:48 PM, riya khanna wrote: We built a system to match mobile app binaries against an indexing database of signatures extracted across various popular OSS sources (e.g., ffmpeg). We should be able to share a draft of our design and implementation with you soon. Thanks! You indicate

Re: [FFmpeg-devel] FFMpeg GPL license Violations by Android apps

2017-05-15 Thread riya khanna
We built a system to match mobile app binaries against an indexing database of signatures extracted across various popular OSS sources (e.g., ffmpeg). We should be able to share a draft of our design and implementation with you soon. Thanks! On Mon, May 15, 2017 at 1:38 PM, Kieran Kunhya wrote:

Re: [FFmpeg-devel] [PATCH] hevc: Fix scaling list prediction delta for the 32x32 inter matrix

2017-05-15 Thread Mark Thompson
On 30/04/17 13:34, Mark Thompson wrote: > Fixes ticket #6356. > --- > This was broken in 627c044f50da3046809314f7cc742b8a10cf26a1, which changed > the array index of the 32x32 inter matrix but missed this case. > > Sample at - with this change the output > f

Re: [FFmpeg-devel] [PATCH] libavformat aviobuf: Fixed dst pointer initialization in fill_buffer

2017-05-15 Thread Rob Meyers
Of course. We noticed when reading data from a named pipe the first 10 bytes would get dropped. I traced this to the affected code in fill_buffer(). The assignment of "dst" was always set to the beginning of the buffer, and if it hadn't been consumed yet the data would be overwritten. We could rep

Re: [FFmpeg-devel] [PATCH] libavformat aviobuf: Fixed dst pointer initialization in fill_buffer

2017-05-15 Thread Hendrik Leppkes
On Mon, May 15, 2017 at 6:47 PM, Rob Meyers wrote: > --- > libavformat/aviobuf.c | 4 +--- > 1 file changed, 1 insertion(+), 3 deletions(-) > Commit messages that say "fixed " are usually not quite informative. Can you elaborate whats the problem, how it exhibits and can be tested and how this i

Re: [FFmpeg-devel] FFMpeg GPL license Violations by Android apps

2017-05-15 Thread Kieran Kunhya
Hi, Out of interest, how did you automate this process? Is it as simple as looking for the FFmpeg configure string? Regards, Kieran Kunhya On Mon, 15 May 2017 at 16:36 riya khanna wrote: > Typo: We have inspected *1.6 million free* Google Playstore Android apps > > On Mon, May 15, 2017 at 11:2

[FFmpeg-devel] [PATCH] libavformat aviobuf: Fixed dst pointer initialization in fill_buffer

2017-05-15 Thread Rob Meyers
--- libavformat/aviobuf.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/libavformat/aviobuf.c b/libavformat/aviobuf.c index 0a7c39eacd..4e04cb79e0 100644 --- a/libavformat/aviobuf.c +++ b/libavformat/aviobuf.c @@ -519,9 +519,7 @@ void avio_write_marker(AVIOContext *s, int6

Re: [FFmpeg-devel] FFMpeg GPL license Violations by Android apps

2017-05-15 Thread riya khanna
Typo: We have inspected *1.6 million free* Google Playstore Android apps On Mon, May 15, 2017 at 11:24 AM, riya khanna wrote: > Please find the list of apps here: > https://docs.google.com/spreadsheets/d/1YnBHbsYIyGxQbjpaIbU4LFjha3hx8 > 3ciKLk7htVjpT4/edit?usp=sharing > > We have inspected 1.6 f

Re: [FFmpeg-devel] FFMpeg GPL license Violations by Android apps

2017-05-15 Thread riya khanna
Please find the list of apps here: https://docs.google.com/spreadsheets/d/1YnBHbsYIyGxQbjpaIbU4LFjha3hx83ciKLk7htVjpT4/edit?usp=sharing We have inspected 1.6 free Google Playstore Android apps and found over 4,300 apps violating GPL 2 or 3 (--enable-gpl) licensing terms. Let us know if you have a

Re: [FFmpeg-devel] [PATCH]Two patches from github

2017-05-15 Thread Ronald S. Bultje
Hi, On Mon, May 15, 2017 at 6:53 AM, wm4 wrote: > On Mon, 15 May 2017 12:40:23 +0200 > Carl Eugen Hoyos wrote: > > Change-Id: I1cbf000a1bda7b3abf0a84e971e752f176857385 > > what is this? Looks like a gerrit (patch review system) patch marker, see e.g.: http://www.webmproject.org/code/contribu

Re: [FFmpeg-devel] [PATCH] Disable MSA optimization for big endian arch

2017-05-15 Thread Ronald S. Bultje
Hi, On Mon, May 15, 2017 at 2:01 AM, Shivraj Patil wrote: > Can anyone review the patch please? > [..] > --- a/configure > +++ b/configure > @@ -5357,6 +5357,10 @@ elif enabled mips; then > enabled mipsdsp && check_inline_asm_flags mipsdsp '"addu.qb $t0, $t1, > $t2"' '-mdsp' > enabled

Re: [FFmpeg-devel] [PATCH]Two patches from github

2017-05-15 Thread wm4
On Mon, 15 May 2017 12:40:23 +0200 Carl Eugen Hoyos wrote: > From 42766f345dbf398716c6fd9072f072f5fa91c940 Mon Sep 17 00:00:00 2001 > From: Steve Kondik > Date: Tue, 16 Dec 2014 01:37:57 -0800 > Subject: [PATCH 2/2] avutil: Use _SC_NPROCESSORS_CONF > > * On most Android devices, CPUs can appea

[FFmpeg-devel] [PATCH]Two patches from github

2017-05-15 Thread Carl Eugen Hoyos
Hi! Attached patches have found their way into a github merge request, I will commit them if nobody objects. Carl Eugen From 16ea42a6337db216a605cc211df88eb6bb3dfda8 Mon Sep 17 00:00:00 2001 From: Arnav Gupta Date: Wed, 10 Feb 2016 05:36:59 +0530 Subject: [PATCH 1/2] libavutil: fix old style fu

Re: [FFmpeg-devel] [PATCH 3/3] avfilter/scale_cuda: add CUDA scale filter

2017-05-15 Thread Timo Rothenpieler
Am 14.05.2017 um 17:36 schrieb Timo Rothenpieler: >>> +AV_PIX_FMT_P016 >>> +}; >>> + >>> +#define DIV_UP(a, b) ( ((a) + (b) - 1) / (b) ) >> >>> +#define ALIGN_UP(a, b) ((a + b -1) & ~(b-1)) >> >> this is missing () to protect the arguments > > > Fixed locally, will push this evening or tomorr

Re: [FFmpeg-devel] [PATCH] videotoolbox: add hwcontext support

2017-05-15 Thread wm4
On Sun, 7 May 2017 12:01:11 -0700 Aaron Levinson wrote: > Are you also planning to change ffmpeg_videotoolbox.c? See below for > more comments. > > Aaron Levinson > Pushed. Changed what I felt like I could change. Thanks for the review. ___ ffmpeg-