[Bug 700491] New: [dashdemux] Handle cases where minimumUpdatePeriod sets the period length

[GStreamer (bugzilla.gnome.org)]

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

           Summary: [dashdemux] Handle cases where minimumUpdatePeriod
                    sets the period length
    Classification: Platform
           Product: GStreamer
           Version: git
        OS/Version: Linux
            Status: UNCONFIRMED
          Severity: normal
          Priority: Normal
         Component: gst-plugins-bad
        AssignedTo: gstreamer-bugs at lists.freedesktop.org
        ReportedBy: art at videon-central.com
         QAContact: gstreamer-bugs at lists.freedesktop.org
                CC: greg at gsr-tek.com
     GNOME version: ---


Created an attachment (id=244472)
 View: https://bugzilla.gnome.org/attachment.cgi?id=244472
 Review: https://bugzilla.gnome.org/review?bug=700491&attachment=244472

Initial patch

On a live ts stream generator I ran into this case; unfortunately it is not
publicly available for reproduction.

Per A.3.2 of ISO/IEC 23009, concerning Representations using SegmentTemplates:

"if the MPD at mediaPresentationDuration attribute is not present, then
PeriodEndTime is defined
as FetchTime + MPD at minimumUpdatePeriod."

However, the code fails out in this case and refuses to even start playback,
since the PeriodEnd value in gst_mpd_client_setup_representation() sets
PeriodEnd to GST_CLOCK_TIME_NONE in such a case because no Duration is
provided.

I have attached a patch to fix this issue. It also provides a comment for
future debugging of similar cases.

-- 
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.