[Bug 750797] New: dashdemux: period duration is not validated

[GStreamer (GNOME Bugzilla)]

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

            Bug ID: 750797
           Summary: dashdemux: period duration is not validated
    Classification: Platform
           Product: GStreamer
           Version: git master
                OS: Linux
            Status: NEW
          Severity: normal
          Priority: Normal
         Component: gst-plugins-bad
          Assignee: gstreamer-bugs at lists.freedesktop.org
          Reporter: florin.apostol at oregan.net
        QA Contact: gstreamer-bugs at lists.freedesktop.org
     GNOME version: ---

gst_mpd_client_setup_media_presentation tries to compute period duration. In
cases when the computation is based on information from next period or from
mediaPresentationDuration, the duration can be set to a negative value if the
information is not correct.

Scenario 1:
      /* try to infer this period duration from the start time of the next
period */
      GstPeriodNode *next_period_node = next->data;
      if (next_period_node->start != -1) {
        duration = next_period_node->start * GST_MSECOND - start;

If next_period_node->start is wrongly configured less than start, duration will
be negative.

Scenario 2:
    } else if (client->mpd_node->mediaPresentationDuration != -1) {
      /* last Period of the Media Presentation */
      duration =
          client->mpd_node->mediaPresentationDuration * GST_MSECOND - start;

If mediaPresentationDuration is less than start, duration will be negative.

duration is stored in GstClockTime which is unsigned, meaning we will end up
with a huge duration.

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