Re: [FFmpeg-devel] [PATCH v2] avformat/mov: abort reading truncated stts

2021-12-22 Thread Gyan Doshi
Patch superseded by new patch using helper function. On 2021-12-21 10:20 am, Gyan Doshi wrote: On 2021-12-21 03:06 am, Michael Niedermayer wrote: On Mon, Dec 20, 2021 at 10:21:53PM +0100, Michael Niedermayer wrote: On Tue, Dec 21, 2021 at 02:31:33AM +0530, Gyan Doshi wrote: On 2021-12-21 0

Re: [FFmpeg-devel] [PATCH v2] avformat/mov: abort reading truncated stts

2021-12-20 Thread Gyan Doshi
On 2021-12-21 03:06 am, Michael Niedermayer wrote: On Mon, Dec 20, 2021 at 10:21:53PM +0100, Michael Niedermayer wrote: On Tue, Dec 21, 2021 at 02:31:33AM +0530, Gyan Doshi wrote: On 2021-12-21 02:24 am, Andreas Rheinhardt wrote: Gyan Doshi: On 2021-12-21 02:18 am, Andreas Rheinhardt wrote

Re: [FFmpeg-devel] [PATCH v2] avformat/mov: abort reading truncated stts

2021-12-20 Thread Michael Niedermayer
On Mon, Dec 20, 2021 at 10:21:53PM +0100, Michael Niedermayer wrote: > On Tue, Dec 21, 2021 at 02:31:33AM +0530, Gyan Doshi wrote: > > > > > > On 2021-12-21 02:24 am, Andreas Rheinhardt wrote: > > > Gyan Doshi: > > > > > > > > On 2021-12-21 02:18 am, Andreas Rheinhardt wrote: > > > > > Gyan Dosh

Re: [FFmpeg-devel] [PATCH v2] avformat/mov: abort reading truncated stts

2021-12-20 Thread Michael Niedermayer
On Tue, Dec 21, 2021 at 02:31:33AM +0530, Gyan Doshi wrote: > > > On 2021-12-21 02:24 am, Andreas Rheinhardt wrote: > > Gyan Doshi: > > > > > > On 2021-12-21 02:18 am, Andreas Rheinhardt wrote: > > > > Gyan Doshi: > > > > > Avoids overreading the box and ingesting absurd values into stts_data >

Re: [FFmpeg-devel] [PATCH v2] avformat/mov: abort reading truncated stts

2021-12-20 Thread Gyan Doshi
On 2021-12-21 02:24 am, Andreas Rheinhardt wrote: Gyan Doshi: On 2021-12-21 02:18 am, Andreas Rheinhardt wrote: Gyan Doshi: Avoids overreading the box and ingesting absurd values into stts_data ---   libavformat/mov.c | 5 +   1 file changed, 5 insertions(+) diff --git a/libavformat/m

Re: [FFmpeg-devel] [PATCH v2] avformat/mov: abort reading truncated stts

2021-12-20 Thread Andreas Rheinhardt
Gyan Doshi: > > > On 2021-12-21 02:18 am, Andreas Rheinhardt wrote: >> Gyan Doshi: >>> Avoids overreading the box and ingesting absurd values into stts_data >>> --- >>>   libavformat/mov.c | 5 + >>>   1 file changed, 5 insertions(+) >>> >>> diff --git a/libavformat/mov.c b/libavformat/mov.c >

Re: [FFmpeg-devel] [PATCH v2] avformat/mov: abort reading truncated stts

2021-12-20 Thread Gyan Doshi
On 2021-12-21 02:18 am, Andreas Rheinhardt wrote: Gyan Doshi: Avoids overreading the box and ingesting absurd values into stts_data --- libavformat/mov.c | 5 + 1 file changed, 5 insertions(+) diff --git a/libavformat/mov.c b/libavformat/mov.c index 2aed6e80ef..5a7209837f 100644 --- a

Re: [FFmpeg-devel] [PATCH v2] avformat/mov: abort reading truncated stts

2021-12-20 Thread Andreas Rheinhardt
Gyan Doshi: > Avoids overreading the box and ingesting absurd values into stts_data > --- > libavformat/mov.c | 5 + > 1 file changed, 5 insertions(+) > > diff --git a/libavformat/mov.c b/libavformat/mov.c > index 2aed6e80ef..5a7209837f 100644 > --- a/libavformat/mov.c > +++ b/libavformat/mov