[Bug 705991] Adding support for DASH common encryption to qtdemux and dashdemux

[GStreamer (GNOME Bugzilla)]

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

--- Comment #154 from Philippe Normand <phil at base-art.net> ---
(In reply to A Ashley from comment #136)
> (In reply to Philippe Normand from comment #128)
> > (In reply to Philippe Normand from comment #127)
> > > Created attachment 304512 [details] [review] [review] [review]
> > > qtdemux: support for cenc auxiliary info parsing outside of moof box
> > > 
> > > When the cenc aux info index is out of moof boundaries, keep track of
> > > it and parse the beginning of the mdat box, before the first sample.
> > 
> > I'm not totally sure this is the right way to do but it worked for the
> > sample I have here. Feedback welcome :)
> 
> I think the DASH CENC specification allows the saio box to point to anywhere
> in the fragment for the CommonEncryptionSampleAuxiliaryInformation. It might
> not be right to assume that it is always before the first sample.
> 

Hum ok. Would you by any chance have some sample files where the saio box
wouldn't be before the first sample?

For this patch I've been using this:
https://github.com/youtube/js_mse_eme/blob/master/media/oops_cenc-20121114-145-no-clear-start.mp4

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