[Bug 776200] dashdemux: Always parsing sidx for On-Demand profile

[GStreamer (GNOME Bugzilla)]

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

Sebastian Dröge (slomo) <slomo at coaxion.net> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|RESOLVED                    |REOPENED
         Resolution|FIXED                       |---

--- Comment #8 from Sebastian Dröge (slomo) <slomo at coaxion.net> ---
(In reply to Seungha Yang from comment #3)
> Some on-demand profile MPDs have no indexRange attribute. Spec. is saying
> that is optional.

While the spec says that it's optional, it says that if the value is not given
then it's "unknown" (for SegmentBase) or there is no index table (for
SegmentList).

I don't see how that means that we should parse any sidx that we might find
while parsing the segment and assume that it is valid for the whole media
segment.


In my case I have a stream here that contains a SegmentList without index, and
the SegmentList works as an index (it has the offsets of each fragment).
However at the beginning of each fragment, there is a sidx that only describes
the following moof. We're now parsing each of them one after another and have
incomplete/unexpected information in the sidx index, which then causes infinite
loops during parsing.

I'd revert these commits again unless you disagree, if anything we will have to
do something more clever here than just blindly parsing any sidx that we might
see somewhere.

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