[Bug 766301] qtdemux: Parsing elst box based on version

[GStreamer (GNOME Bugzilla)]

https://bugzilla.gnome.org/show_bug.cgi?id=766301

--- Comment #12 from Seungha Yang <sh.yang at lge.com> ---
(In reply to Sebastian Dröge (slomo) from comment #10)
> Seungha Yang, are you preparing a patch for the test? The fix itself is
> going to be in 1.8.2 and 1.9.1

I upload a patch for qtmux (bug #766760). However, that patch is just to modify
variables from 32bit to 64bit (elst). 
When I saw qtmux code, version of most atoms are fixed to "version 0"... 
So, I think that current qtmux does not consider overflow of duration. Not only
elst, but other boxes such as mvhd, mdhd, and trak can have 64bit and in order
to use it, the version should be "version 1".

What I'm going to do is
- 1. Apply patch bug #766760 which can give a chance to support "version 1"
elst. (but muxing using "version 1" boxes is not implemented yet in qtmux)
- 2. Adding another feature in qtmux to support "version 1" boxes based on
checked duration (whether it overs G_MAX_UINT32 or not), or new properties...
with test code.

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.