Re: [FFmpeg-devel] [PATCH] avcodec/vp9block: fix runtime error: signed integer overflow: 196675 * 20670 cannot be represented in type 'int'

2017-05-21 Thread Michael Niedermayer
On Sat, May 20, 2017 at 10:35:34PM -0400, Ronald S. Bultje wrote: > Hi, > > On Sat, May 20, 2017 at 8:12 PM, Michael Niedermayer > wrote: > > > Fixes: 1710/clusterfuzz-testcase-minimized-4837032931098624 > > > > Found-by: continuous fuzzing process https://github.com/google/oss- > > fuzz/tree/ma

Re: [FFmpeg-devel] [PATCH] avcodec/wavpack: Fix multiple runtime error: signed integer overflow: 548 * -2147483648 cannot be represented in type 'int'

2017-05-21 Thread Michael Niedermayer
On Thu, May 18, 2017 at 03:54:29AM +0200, Michael Niedermayer wrote: > Fixes: 1659/clusterfuzz-testcase-minimized-5396490639900672 > > Found-by: continuous fuzzing process > https://github.com/google/oss-fuzz/tree/master/targets/ffmpeg > Signed-off-by: Michael Niedermayer > --- > libavcodec/wav

Re: [FFmpeg-devel] [PATCH v2] avfilter: take_samples: do not directly return frame when samples are skipped

2017-05-21 Thread wm4
On Sat, 20 May 2017 20:12:15 +0200 Nicolas George wrote: > Le primidi 1er prairial, an CCXXV, Muhammad Faiz a écrit : > > > I will push this soon. > > Pushed and backported to 3.3 branch. > > Well, I was probably ok with this patch, but we will never know, will > we? > > Can you explain the

Re: [FFmpeg-devel] [PATCH] avcodec/takdec: Fix runtime error: signed integer overflow: 8192 * 524308 cannot be represented in type 'int'

2017-05-21 Thread Michael Niedermayer
On Wed, May 17, 2017 at 12:31:18AM +0200, Michael Niedermayer wrote: > Fixes: 1630/clusterfuzz-testcase-minimized-6326111917047808 > > Found-by: continuous fuzzing process > https://github.com/google/oss-fuzz/tree/master/targets/ffmpeg > Signed-off-by: Michael Niedermayer applied [...] -- Mic

Re: [FFmpeg-devel] [PATCH] avcodec/vp9block: fix runtime error: signed integer overflow: 196675 * 20670 cannot be represented in type 'int'

2017-05-21 Thread Michael Niedermayer
On Sun, May 21, 2017 at 12:51:56PM +0200, Michael Niedermayer wrote: > On Sat, May 20, 2017 at 10:35:34PM -0400, Ronald S. Bultje wrote: > > Hi, > > > > On Sat, May 20, 2017 at 8:12 PM, Michael Niedermayer > > wrote: > > > > > Fixes: 1710/clusterfuzz-testcase-minimized-4837032931098624 > > > > >

Re: [FFmpeg-devel] [PATCH] avcodec/vp9block: fix runtime error: signed integer overflow: 196675 * 20670 cannot be represented in type 'int'

2017-05-21 Thread Ronald S. Bultje
Hi, On Sun, May 21, 2017 at 6:57 AM, Michael Niedermayer wrote: > On Sun, May 21, 2017 at 12:51:56PM +0200, Michael Niedermayer wrote: > > On Sat, May 20, 2017 at 10:35:34PM -0400, Ronald S. Bultje wrote: > > > Hi, > > > > > > On Sat, May 20, 2017 at 8:12 PM, Michael Niedermayer > > > > wrote:

Re: [FFmpeg-devel] [PATCH 3/3] avcodec/mlpdec: Check quant_step_size against huff_lsbs

2017-05-21 Thread wm4
On Sat, 20 May 2017 23:01:04 +0200 Michael Niedermayer wrote: > This reorders the operations so as to avoid computations with the above > arguments > before they have been initialized. > Fixes part of 1708/clusterfuzz-testcase-minimized-5035111957397504 > > Found-by: continuous fuzzing process

[FFmpeg-devel] [PATCH] avcodec/tscc2: Skip duplicate frames

2017-05-21 Thread Michael Niedermayer
This turns CFR duplicated frames into skiped frames Fixes: Timeout Fixes: 1719/clusterfuzz-testcase-minimized-6375090079924224 Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg Signed-off-by: Michael Niedermayer --- libavcodec/tscc2.c | 1

Re: [FFmpeg-devel] [PATCH] avcodec/vp9block: fix runtime error: signed integer overflow: 196675 * 20670 cannot be represented in type 'int'

2017-05-21 Thread Michael Niedermayer
On Sun, May 21, 2017 at 07:25:10AM -0400, Ronald S. Bultje wrote: > Hi, > > On Sun, May 21, 2017 at 6:57 AM, Michael Niedermayer > wrote: > > > On Sun, May 21, 2017 at 12:51:56PM +0200, Michael Niedermayer wrote: > > > On Sat, May 20, 2017 at 10:35:34PM -0400, Ronald S. Bultje wrote: > > > > Hi,

[FFmpeg-devel] [PATCH 1/3] lavc/mediacodec_wrapper: fix local reference leaks

2017-05-21 Thread Matthieu Bouron
--- libavcodec/mediacodec_wrapper.c | 37 ++--- 1 file changed, 26 insertions(+), 11 deletions(-) diff --git a/libavcodec/mediacodec_wrapper.c b/libavcodec/mediacodec_wrapper.c index 43fbb511fd..5e1beed43a 100644 --- a/libavcodec/mediacodec_wrapper.c +++ b/libavcod

[FFmpeg-devel] [PATCH 2/2] lavc/ffjni: add missing '\n'

2017-05-21 Thread Matthieu Bouron
--- libavcodec/ffjni.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavcodec/ffjni.c b/libavcodec/ffjni.c index b2bcae9602..f5b581f0f6 100644 --- a/libavcodec/ffjni.c +++ b/libavcodec/ffjni.c @@ -85,7 +85,7 @@ JNIEnv *ff_jni_get_env(void *log_ctx) av_log(log_ctx,

[FFmpeg-devel] [PATCH 1/2] lavc/mediacodec_wrapper: do not declare JNIAMedia{Codec, Format}Fields on the stack

2017-05-21 Thread Matthieu Bouron
--- libavcodec/mediacodec_wrapper.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/libavcodec/mediacodec_wrapper.c b/libavcodec/mediacodec_wrapper.c index 5e1beed43a..0f573bfc7a 100644 --- a/libavcodec/mediacodec_wrapper.c +++ b/libavcodec/mediacodec_wrapper.c @@ -125,7 +1

Re: [FFmpeg-devel] [PATCH 2/2] lavc/ffjni: add missing '\n'

2017-05-21 Thread Carl Eugen Hoyos
2017-05-21 18:17 GMT+02:00 Matthieu Bouron : > -av_log(log_ctx, AV_LOG_ERROR, "Failed to get the JNI environment > attached to this thread"); > +av_log(log_ctx, AV_LOG_ERROR, "Failed to get the JNI environment > attached to this thread\n"); Please commit such fixes without sending

Re: [FFmpeg-devel] [PATCH 2/2] lavc/ffjni: add missing '\n'

2017-05-21 Thread Matthieu Bouron
On Sun, May 21, 2017 at 06:36:10PM +0200, Carl Eugen Hoyos wrote: > 2017-05-21 18:17 GMT+02:00 Matthieu Bouron : > > > -av_log(log_ctx, AV_LOG_ERROR, "Failed to get the JNI environment > > attached to this thread"); > > +av_log(log_ctx, AV_LOG_ERROR, "Failed to get the JNI environm

Re: [FFmpeg-devel] [PATCH 1/2] lavc/mediacodec_wrapper: do not declare JNIAMedia{Codec, Format}Fields on the stack

2017-05-21 Thread Matthieu Bouron
On Sun, May 21, 2017 at 06:17:27PM +0200, Matthieu Bouron wrote: > --- > libavcodec/mediacodec_wrapper.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/libavcodec/mediacodec_wrapper.c b/libavcodec/mediacodec_wrapper.c > index 5e1beed43a..0f573bfc7a 100644 > --- a/lib

[FFmpeg-devel] [PATCH] avcodec/aacps: Check border_position to be monotone

2017-05-21 Thread Michael Niedermayer
Fixes: runtime error: left shift of negative value -67108864 Fixes: 1738/clusterfuzz-testcase-minimized-6734814327603200 Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg Signed-off-by: Michael Niedermayer --- libavcodec/aacps.c | 7 ++- 1 f

Re: [FFmpeg-devel] [PATCH]lavf/img2: Allow reading svg image sequences

2017-05-21 Thread Carl Eugen Hoyos
2017-05-17 15:39 GMT+02:00 Rostislav Pehlivanov : > On 17 May 2017 at 13:58, Carl Eugen Hoyos wrote: > >> 2017-05-17 14:46 GMT+02:00 Rostislav Pehlivanov : >> > On 17 May 2017 at 10:03, Carl Eugen Hoyos wrote: >> > >> >> Hi! >> >> >> >> I believe attached patch is necessary to allow reading svg s

[FFmpeg-devel] question on mosiac video issues -shortest

2017-05-21 Thread Kyle Douglas
When using the param -shortest appears to have no impact on my ffmpeg command using multiple inputs as source. One is a looping slate video that stops and it seems to stop all other inputs when making a mp4 archive. Any suggestions on what I can do as using another param or output syntax? 3 exampl

Re: [FFmpeg-devel] [PATCH V3] lavc/vaapi_encode_h264: Enable MB rate control.

2017-05-21 Thread Jun Zhao
On 2017/5/20 20:32, Mark Thompson wrote: > On 19/05/17 14:10, Mark Thompson wrote: >> On 19/05/17 00:43, Jun Zhao wrote: >>> >>> On 2017/5/14 12:26, Jun Zhao wrote: V3: - Fix build error with old VAAPI version. V2: - Refine the name/value type to mb_rate_control/bool. - Only su

Re: [FFmpeg-devel] [PATCH]libavformat/http: fix http error eof

2017-05-21 Thread raymond zheng
ping... 2017-05-18 15:19 GMT+08:00 raymond zheng : > Hi: > I find an issue about http. I don't use chunked, so s->chunksize will > be set as UINT64_MAX when http open, but because of "if (s->chunksize > > 0) s->chunksize -= len;" then chunksize will not be UINT64_MAX. > > If ffurl_read re