[FFmpeg-devel] [PATCH] lavf/mov: ignore ctts entries that do not apply to a least one sample

[Michael Niedermayer]

On Thu, Jun 16, 2016 at 05:26:14PM +0200, Matthieu Bouron wrote:
> From: Matthieu Bouron <matthieu.bouron at stupeflix.com>
> 
> Fixes packet pts of samples which contain ctts entries with count=0.
> ---
> 
> Hello,
> 
> The following patch fixes packet pts of samples which contain ctts values with
> count=0 (so the ctts entry does not apply to any sample if I understand
> correctly). Such samples are produced by a LG G4 phone. I don't have any
> sample I can share at the moment (and thus no fate test following this patch
> yet).
> 
> An alternative to this patch is to remove directly the entry when the ctts atom
> is parsed. Would you prefer this alternative ?

i dont know what is preferred but i agree about either solution

removing them on load would avoid any issues with ctts_count > 0
and no real entries, i dont know though if that ever matters

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

No snowflake in an avalanche ever feels responsible. -- Voltaire
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 181 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20160617/1839d8dd/attachment.sig>