Re: [FFmpeg-devel] Fix memset size on ctts_data in mov_read_trun()

2018-03-01 Thread Michael Niedermayer
On Tue, Feb 27, 2018 at 01:11:56PM -0800, Xiaohan Wang (王消寒) wrote: > Sure. Updated! > > On Tue, Feb 27, 2018 at 2:38 AM, Michael Niedermayer > wrote: > > > On Mon, Feb 26, 2018 at 10:37:51AM -0800, Xiaohan Wang (王消寒) wrote: > > > Thanks! Updated the patch. Please take a look again. > > > > > >

Re: [FFmpeg-devel] Fix memset size on ctts_data in mov_read_trun()

2018-02-27 Thread 王消寒
Sure. Updated! On Tue, Feb 27, 2018 at 2:38 AM, Michael Niedermayer wrote: > On Mon, Feb 26, 2018 at 10:37:51AM -0800, Xiaohan Wang (王消寒) wrote: > > Thanks! Updated the patch. Please take a look again. > > > > On Sat, Feb 24, 2018 at 7:04 PM, Michael Niedermayer > > > wrote: > > > > > On Fri, F

Re: [FFmpeg-devel] Fix memset size on ctts_data in mov_read_trun()

2018-02-27 Thread Michael Niedermayer
On Mon, Feb 26, 2018 at 10:37:51AM -0800, Xiaohan Wang (王消寒) wrote: > Thanks! Updated the patch. Please take a look again. > > On Sat, Feb 24, 2018 at 7:04 PM, Michael Niedermayer > wrote: > > > On Fri, Feb 23, 2018 at 05:12:05PM -0800, Xiaohan Wang (王消寒) wrote: > > > Michael: Dale and I dig int

Re: [FFmpeg-devel] Fix memset size on ctts_data in mov_read_trun()

2018-02-26 Thread 王消寒
Thanks! Updated the patch. Please take a look again. On Sat, Feb 24, 2018 at 7:04 PM, Michael Niedermayer wrote: > On Fri, Feb 23, 2018 at 05:12:05PM -0800, Xiaohan Wang (王消寒) wrote: > > Michael: Dale and I dig into history a bit more and we don't understand > why > > the code is changed to the

Re: [FFmpeg-devel] Fix memset size on ctts_data in mov_read_trun()

2018-02-24 Thread Michael Niedermayer
On Fri, Feb 23, 2018 at 05:12:05PM -0800, Xiaohan Wang (王消寒) wrote: > Michael: Dale and I dig into history a bit more and we don't understand why > the code is changed to the current state around memset. This new patch > reverted the change back to the previous state which we felt is much > cleaner

Re: [FFmpeg-devel] Fix memset size on ctts_data in mov_read_trun()

2018-02-23 Thread 王消寒
Michael: Dale and I dig into history a bit more and we don't understand why the code is changed to the current state around memset. This new patch reverted the change back to the previous state which we felt is much cleaner. Please see the CL description for details and take a look at the new patch

Re: [FFmpeg-devel] Fix memset size on ctts_data in mov_read_trun()

2018-02-21 Thread 王消寒
jstebbins: kindly ping! On Fri, Feb 16, 2018 at 2:42 PM, Xiaohan Wang (王消寒) wrote: > +jstebbins@ who wrote that code. > > On Fri, Feb 16, 2018 at 12:30 PM, Michael Niedermayer < > mich...@niedermayer.cc> wrote: > >> On Thu, Feb 15, 2018 at 12:10:33PM -0800, Xiaohan Wang (王消寒) wrote: >> > >> >> >

Re: [FFmpeg-devel] Fix memset size on ctts_data in mov_read_trun()

2018-02-16 Thread 王消寒
+jstebbins@ who wrote that code. On Fri, Feb 16, 2018 at 12:30 PM, Michael Niedermayer < mich...@niedermayer.cc> wrote: > On Thu, Feb 15, 2018 at 12:10:33PM -0800, Xiaohan Wang (王消寒) wrote: > > > > > mov.c |3 ++- > > 1 file changed, 2 insertions(+), 1 deletion(-) > > 5597d0b095f8b15eb115030

Re: [FFmpeg-devel] Fix memset size on ctts_data in mov_read_trun()

2018-02-16 Thread Michael Niedermayer
On Thu, Feb 15, 2018 at 12:10:33PM -0800, Xiaohan Wang (王消寒) wrote: > > mov.c |3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > 5597d0b095f8b15eb11503010a51c2bc2c022413 > 0001-ffmpeg-Fix-memset-size-on-ctts_data-in-mov_read_trun.patch > From 7c1e6b50ebe35b2a38c4f1d0a988e31eccbd0ea

[FFmpeg-devel] Fix memset size on ctts_data in mov_read_trun()

2018-02-15 Thread 王消寒
From 7c1e6b50ebe35b2a38c4f1d0a988e31eccbd0ead Mon Sep 17 00:00:00 2001 From: Xiaohan Wang Date: Thu, 15 Feb 2018 12:05:53 -0800 Subject: [PATCH] ffmpeg: Fix memset size on ctts_data in mov_read_trun() The allocated size of sc->ctts_data is (st->nb_index_entries + entries) * sizeof(*sc->ctts_data