Re: [FFmpeg-devel] [PATCH] avformat/mxfdec: Check index_duration

2023-01-29 Thread Marton Balint
On Sun, 22 Jan 2023, Marton Balint wrote: On Wed, 28 Dec 2022, Marton Balint wrote: On Tue, 27 Dec 2022, Marton Balint wrote: On Tue, 27 Dec 2022, Michael Niedermayer wrote: On Tue, Dec 27, 2022 at 07:05:44PM +0100, Marton Balint wrote: On Mon, 26 Dec 2022, Michael Nied

Re: [FFmpeg-devel] [PATCH] avformat/mxfdec: Check index_duration

2023-01-21 Thread Marton Balint
On Wed, 28 Dec 2022, Marton Balint wrote: On Tue, 27 Dec 2022, Marton Balint wrote: On Tue, 27 Dec 2022, Michael Niedermayer wrote: On Tue, Dec 27, 2022 at 07:05:44PM +0100, Marton Balint wrote: On Mon, 26 Dec 2022, Michael Niedermayer wrote: On Mon, Dec 26, 2022 at 11:32:4

Re: [FFmpeg-devel] [PATCH] avformat/mxfdec: Check index_duration

2022-12-27 Thread Marton Balint
On Tue, 27 Dec 2022, Marton Balint wrote: On Tue, 27 Dec 2022, Michael Niedermayer wrote: On Tue, Dec 27, 2022 at 07:05:44PM +0100, Marton Balint wrote: On Mon, 26 Dec 2022, Michael Niedermayer wrote: On Mon, Dec 26, 2022 at 11:32:48AM +0100, Tomas Härdin wrote: lör 2022-12-24 kl

Re: [FFmpeg-devel] [PATCH] avformat/mxfdec: Check index_duration

2022-12-27 Thread Tomas Härdin
tis 2022-12-27 klockan 22:49 +0100 skrev Michael Niedermayer: > On Tue, Dec 27, 2022 at 07:05:44PM +0100, Marton Balint wrote: > > > > > > On Mon, 26 Dec 2022, Michael Niedermayer wrote: > > > > > On Mon, Dec 26, 2022 at 11:32:48AM +0100, Tomas Härdin wrote: > > > > lör 2022-12-24 klockan 23:50

Re: [FFmpeg-devel] [PATCH] avformat/mxfdec: Check index_duration

2022-12-27 Thread Marton Balint
On Tue, 27 Dec 2022, Michael Niedermayer wrote: On Tue, Dec 27, 2022 at 07:05:44PM +0100, Marton Balint wrote: On Mon, 26 Dec 2022, Michael Niedermayer wrote: On Mon, Dec 26, 2022 at 11:32:48AM +0100, Tomas Härdin wrote: lör 2022-12-24 klockan 23:50 +0100 skrev Michael Niedermayer:    

Re: [FFmpeg-devel] [PATCH] avformat/mxfdec: Check index_duration

2022-12-27 Thread Michael Niedermayer
On Tue, Dec 27, 2022 at 07:05:44PM +0100, Marton Balint wrote: > > > On Mon, 26 Dec 2022, Michael Niedermayer wrote: > > > On Mon, Dec 26, 2022 at 11:32:48AM +0100, Tomas Härdin wrote: > > > lör 2022-12-24 klockan 23:50 +0100 skrev Michael Niedermayer: > > > > > > > > index_table->nb_p

Re: [FFmpeg-devel] [PATCH] avformat/mxfdec: Check index_duration

2022-12-27 Thread Marton Balint
On Mon, 26 Dec 2022, Michael Niedermayer wrote: On Mon, Dec 26, 2022 at 11:32:48AM +0100, Tomas Härdin wrote: lör 2022-12-24 klockan 23:50 +0100 skrev Michael Niedermayer: index_table->nb_ptses += s->index_duration; +    // If index_duration is substantially larger than nb_inde

Re: [FFmpeg-devel] [PATCH] avformat/mxfdec: Check index_duration

2022-12-26 Thread Michael Niedermayer
On Mon, Dec 26, 2022 at 11:32:48AM +0100, Tomas Härdin wrote: > lör 2022-12-24 klockan 23:50 +0100 skrev Michael Niedermayer: > > > > index_table->nb_ptses += s->index_duration; > > +    // If index_duration is substantially larger than > > nb_index_entries then this algorithm which >

Re: [FFmpeg-devel] [PATCH] avformat/mxfdec: Check index_duration

2022-12-26 Thread Tomas Härdin
lör 2022-12-24 klockan 23:50 +0100 skrev Michael Niedermayer: > > index_table->nb_ptses += s->index_duration; > +    // If index_duration is substantially larger than > nb_index_entries then this algorithm which > +    // allocates index_duration elements is a bad idea. All > file

[FFmpeg-devel] [PATCH] avformat/mxfdec: Check index_duration

2022-12-24 Thread Michael Niedermayer
Fixes: OOM Fixes: 50551/clusterfuzz-testcase-minimized-ffmpeg_dem_MXF_fuzzer-6607795234930688 Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg Signed-off-by: Michael Niedermayer --- libavformat/mxfdec.c | 4 1 file changed, 4 insertions(+