[Bug 751103] qtdemux: Segment seeks broken with CTTS version 1 .mov files

[GStreamer (GNOME Bugzilla)]

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

--- Comment #18 from Nicolas Dufresne (stormer) <nicolas.dufresne at collabora.co.uk> ---
(In reply to Thiago Sousa Santos from comment #17)
> Review of attachment 305629 [details] [review]:
> 
> This is very close to a solution but there are still some things to solve.
> 
> ::: gst/isomp4/qtdemux.c
> @@ +7185,3 @@
> +
> +    stream->cslg_shift = ABS (least_soffset);
> +  }
> 
> Also, keep in mind that parse_samples will be called incrementally, parsing
> only the first N samples so the observation of least_offset might not be
> correct at the beginning, this means that the initial segment.start could be
> wrong and we could end up with some desync if we find a smaller cslg_shift
> at some point.

We can try and improve the fallback, what do you propose ? Do we parse
incrementally to distribute the load, or will the data be found later in the
stream ?

> 
> Also, we should reset this value when a new fragment starts as it is only
> valid for the movie it is present and not for subsequent ones in a
> fragmented stream.

This is not a new bug, the CTTS has never been reset on streams since day one
of CTTS support.

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