[Bug 674536] tsdemux: Freeze on pts-wrap with streaming sources

[GStreamer (bugzilla.gnome.org)]

https://bugzilla.gnome.org/show_bug.cgi?id=674536
  GStreamer | gst-plugins-bad | git

--- Comment #17 from Holger Kaelberer <hk at getslash.de> 2012-06-26 11:36:27 UTC ---
Created an attachment (id=217275)
 View: https://bugzilla.gnome.org/attachment.cgi?id=217275
 Review: https://bugzilla.gnome.org/review?bug=674536&attachment=217275

Handle resets of PCR/PTS < PCR_GST_MAX_VALUE

We ran into the same issue now: Freeze on resetted PCR/PTS in a multicast
stream containing a looped video. This is a common use-case for public screens
with stuff like commercial content or so and should be handled. (Works fine
with mpegtsdemux/flutsdemux ;-)

The attached patch fixes the issue for our test-cases by kind of generalizing
the existing detection of PCR wraps and PTS rollover for jumps <
PCR_GST_MAX_VALUE (PCR) and PTS_DTS_MAX_VALUE (PTS), resp. 

Do you think this is the way to go? 

Possible that I missed other use-cases! The threshold is now set to 10 sec,
which is ok for our test-content. Is that reasonable? I am not 100% sure,
whether the calculation of the >pts_reset_diff value is correct in all the
cases. 

Thanks for your feedback!

-- 
Configure bugmail: https://bugzilla.gnome.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug.
You are the assignee for the bug.