Re: [FFmpeg-devel] [PATCH] http: support retry on connection error

2020-11-07 Thread Eran Kornblau
Pinging again... Thanks, Eran From: Eran Kornblau Sent: Sunday, October 25, 2020 3:40 PM To: FFmpeg development discussions and patches mailto:ffmpeg-devel@ffmpeg.org>> Subject: [PATCH] http: support retry on connection error Hi, This patch adds 2 options to http: - reconnect_on_status - a

Re: [FFmpeg-devel] [PATCH 5/5] avcodec/cfhd: check peak.offset so it stays within the 32bit range

2020-11-07 Thread Andreas Rheinhardt
Michael Niedermayer: > Fixes: signed integer overflow: -2147483648 - 4 cannot be represented in type > 'int' > Fixes: > 26907/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_CFHD_fuzzer-5746202330267648 > > Found-by: continuous fuzzing process >

[FFmpeg-devel] [PATCH 3/5] avformat/utils: wrap_timestamp() is only needed for less than 64 bits

2020-11-07 Thread Michael Niedermayer
Fixes: shift exponent 64 is too large for 64-bit type 'unsigned long long' Fixes: 26497/clusterfuzz-testcase-minimized-ffmpeg_dem_AVI_fuzzer-5690188355076096 Fixes: 26903/clusterfuzz-testcase-minimized-ffmpeg_dem_LUODAT_fuzzer-5641466929741824 Found-by: continuous fuzzing process

[FFmpeg-devel] [PATCH 2/5] avformat/id3v2: Sanity check tlen before alloc and uncompress

2020-11-07 Thread Michael Niedermayer
Fixes: Timeout (>20sec -> 65ms) Fixes: 26896/clusterfuzz-testcase-minimized-ffmpeg_dem_DAUD_fuzzer-5691024049176576 Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg Signed-off-by: Michael Niedermayer --- libavformat/id3v2.c | 3 +++ 1 file

[FFmpeg-devel] [PATCH 5/5] avcodec/cfhd: check peak.offset so it stays within the 32bit range

2020-11-07 Thread Michael Niedermayer
Fixes: signed integer overflow: -2147483648 - 4 cannot be represented in type 'int' Fixes: 26907/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_CFHD_fuzzer-5746202330267648 Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg Signed-off-by:

[FFmpeg-devel] [PATCH 4/5] avformat/lvfdec: Check stream_index before use

2020-11-07 Thread Michael Niedermayer
Fixes: assertion failure Fixes: 26905/clusterfuzz-testcase-minimized-ffmpeg_dem_LVF_fuzzer-5724267599364096.fuzz Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg Signed-off-by: Michael Niedermayer --- libavformat/lvfdec.c | 6 +- 1 file

[FFmpeg-devel] [PATCH 1/5] avformat/tedcaptionsdec: Check for overflow in parse_int()

2020-11-07 Thread Michael Niedermayer
Fixes: signed integer overflow: 111 * 10 cannot be represented in type 'long' Fixes: 26892/clusterfuzz-testcase-minimized-ffmpeg_dem_TEDCAPTIONS_fuzzer-5756045055754240 Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg

Re: [FFmpeg-devel] [PATCH] Ticket #8750 Add inline function for the vec_xl intrinsic in non-VSX environments

2020-11-07 Thread Andriy Gelman
On Sat, 31. Oct 10:17, Andriy Gelman wrote: > On Fri, 16. Oct 00:02, Andriy Gelman wrote: > > On Fri, 09. Oct 20:17, Andriy Gelman wrote: > > > From: Chip Kerchner > > > > > > --- > > > libswscale/ppc/yuv2rgb_altivec.c | 10 ++ > > > 1 file changed, 10 insertions(+) > > > > > > diff

Re: [FFmpeg-devel] [PATCH 5/8] tools/target_dem_fuzzer: Consider it an EIO when reading position wraps around 64bit

2020-11-07 Thread Michael Niedermayer
On Sun, Oct 25, 2020 at 12:23:09AM +0200, Michael Niedermayer wrote: > Fixes: signed integer overflow: 9223372036854775807 + 564 cannot be > represented in type 'long' > Fixes: > 26494/clusterfuzz-testcase-minimized-ffmpeg_dem_VOC_fuzzer-576754158849228 > Fixes: >

Re: [FFmpeg-devel] [PATCH v4] Unbreak av_malloc_max(0) API/ABI

2020-11-07 Thread Ronald S. Bultje
Hi, On Fri, Nov 6, 2020 at 11:10 AM Joakim Tjernlund < joakim.tjernl...@infinera.com> wrote: > We unbundle the ffmpeg/mesa and use system mesa/ffmpeg That's the answer to my earlier question also, thank you. I'm fine with applying this, I would suggest that the "new" (correct) behaviour should

Re: [FFmpeg-devel] [PATCH 5/7] avformat/mpegts: Limit copied data to space

2020-11-07 Thread Michael Niedermayer
On Wed, Nov 04, 2020 at 11:17:53PM +0100, Marton Balint wrote: > > > On Wed, 4 Nov 2020, Michael Niedermayer wrote: > > > Fixes: out of array access > > Fixes: > > 26816/clusterfuzz-testcase-minimized-ffmpeg_dem_MPEGTSRAW_fuzzer-6282861159907328.fuzz > > > > Found-by: continuous fuzzing

Re: [FFmpeg-devel] [PATCH 1/2] avcodec/gdv: Remove dead check

2020-11-07 Thread Paul B Mahol
lgtm On Sat, Nov 7, 2020 at 2:30 AM Andreas Rheinhardt < andreas.rheinha...@gmail.com> wrote: > At the end of its decode function, the decoder sets *got_frame to 1 and > then checks whether ret is < 0; if so, it is returned, otherwise > avpkt->size is. But it is impossible for ret to be < 0 here

Re: [FFmpeg-devel] [PATCH 1/2] Revert "avcodec/adpcm_swf: support decoding multiple fixed-sized blocks at once"

2020-11-07 Thread Zane van Iperen
Is incorrect behaviour. Was covering for an encoder bug where it produced frames of the wrong size. Fixes: out of array write Fixes: 26821/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_ADPCM_SWF_fuzzer-5764465137811456 Found-by: continuous fuzzing process