[FFmpeg-devel] [PATCH] lavf/mov: make invalid mdhd time_scale default to 1 instead of erroring out

[Matthieu Bouron]

On Wed, May 17, 2017 at 01:56:13PM +0200, Matthieu Bouron wrote:
> On Fri, May 12, 2017 at 11:12:12PM +0200, Michael Niedermayer wrote:
> > On Thu, May 11, 2017 at 04:33:50PM +0200, Matthieu Bouron wrote:
> > > Some samples have their metadata track time_scale incorrectly set to 0
> > > and the check introduced by a398f054fdb9b0f0b5a91c231fba6ce014143f71
> > > prevents playback of those samples. Setting the time_scale to 1 fixes
> > > playback.
> > > ---
> > >  libavformat/mov.c | 4 ++--
> > >  1 file changed, 2 insertions(+), 2 deletions(-)
> > 
> > should be ok
> 
> Do you agree if I extend the patch to apply this behaviour to the mvhd
> atoms (like a398f054fdb9b0f0b5a91c231fba6ce014143f71 originally did) ?

Here is an updated version of the patch (which extends the behaviour to
the mvhd atoms).

-- 
Matthieu B.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-lavf-mov-make-invalid-m-d-v-hd-time_scale-default-to.patch
Type: text/x-diff
Size: 1797 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20170517/18868923/attachment.patch>