Re: [FFmpeg-devel] [PATCH] lavf/mov: fix huge alloc in mov_read_ctts

2017-11-30 Thread Michael Niedermayer
On Sun, Nov 26, 2017 at 07:32:30AM -0800, John Stebbins wrote: > An invalid file may cause huge alloc. Delay expansion of ctts entries > until the number of samples is known in mov_build_index. > > Found-by: zhao dongzhuo, AD-lab of Venustech > --- > libavformat/mov.c | 31

[FFmpeg-devel] [PATCH] lavf/mov: fix huge alloc in mov_read_ctts

2017-11-26 Thread John Stebbins
An invalid file may cause huge alloc. Delay expansion of ctts entries until the number of samples is known in mov_build_index. Found-by: zhao dongzhuo, AD-lab of Venustech --- libavformat/mov.c | 31 +++ 1 file changed, 27 insertions(+), 4 deletions(-) diff --git

Re: [FFmpeg-devel] [PATCH] lavf/mov: fix huge alloc in mov_read_ctts

2017-11-26 Thread Derek Buitenhuis
On 11/26/2017 3:10 PM, John Stebbins wrote: > Is there some git magic for this, or is this just something you add manually > to the bottom of the commit message? It's just manual. - Derek ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org

Re: [FFmpeg-devel] [PATCH] lavf/mov: fix huge alloc in mov_read_ctts

2017-11-26 Thread John Stebbins
On 11/25/2017 05:03 PM, James Almer wrote: > On 11/25/2017 10:00 PM, John Stebbins wrote: >> On 11/25/2017 04:03 PM, Carl Eugen Hoyos wrote: >>> 2017-11-25 21:11 GMT+01:00 John Stebbins : An invalid file may cause huge alloc. Delay expansion of ctts entries

Re: [FFmpeg-devel] [PATCH] lavf/mov: fix huge alloc in mov_read_ctts

2017-11-25 Thread James Almer
On 11/25/2017 10:00 PM, John Stebbins wrote: > On 11/25/2017 04:03 PM, Carl Eugen Hoyos wrote: >> 2017-11-25 21:11 GMT+01:00 John Stebbins : >>> An invalid file may cause huge alloc. Delay expansion of ctts entries >>> until the number of samples is known in

Re: [FFmpeg-devel] [PATCH] lavf/mov: fix huge alloc in mov_read_ctts

2017-11-25 Thread John Stebbins
On 11/25/2017 04:03 PM, Carl Eugen Hoyos wrote: > 2017-11-25 21:11 GMT+01:00 John Stebbins : >> An invalid file may cause huge alloc. Delay expansion of ctts entries >> until the number of samples is known in mov_build_index. > Please mention zhao dongzhuo from ADlab of

Re: [FFmpeg-devel] [PATCH] lavf/mov: fix huge alloc in mov_read_ctts

2017-11-25 Thread Carl Eugen Hoyos
2017-11-25 21:11 GMT+01:00 John Stebbins : > An invalid file may cause huge alloc. Delay expansion of ctts entries > until the number of samples is known in mov_build_index. Please mention zhao dongzhuo from ADlab of Venustech who found this issue, I can confirm that

[FFmpeg-devel] [PATCH] lavf/mov: fix huge alloc in mov_read_ctts

2017-11-25 Thread John Stebbins
An invalid file may cause huge alloc. Delay expansion of ctts entries until the number of samples is known in mov_build_index. --- libavformat/mov.c | 31 +++ 1 file changed, 27 insertions(+), 4 deletions(-) diff --git a/libavformat/mov.c b/libavformat/mov.c index