[Bug 752337] New: dashdemux: invalid periods are accepted by the parser

[GStreamer (GNOME Bugzilla)]

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

            Bug ID: 752337
           Summary: dashdemux: invalid periods are accepted by the parser
    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: ---

According to the description of the xs:duration xml datatype
(http://books.xmlschemata.org/relaxng/ch19-77073.html) the format for duration
mandates that the year precedes the month, eg: P1M2Y is invalid (parts order is
significant and Y must precede M).

However, the implementation of gst_mpdparser_get_xml_prop_duration does not
care about the order of these and will accept a P1M2Y duration. More, because
of the algorithm logic, it will also accept a P1Y2Y duration (the last value
will override a previous one).

gst_mpdparser_get_xml_prop_duration implementation should be strengthen to
reject invalid formats.

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