[Bug 752333] New: dashdemux: durations should be validated

[GStreamer (GNOME Bugzilla)]

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

            Bug ID: 752333
           Summary: dashdemux: durations should be 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: ---

The following fields contain duration values and should be checked that they
contain only positive values:
GstPeriodNode start
GstPeriodNode duration
GstMetricsRangeNode startTime
GstMetricsRangeNode duration
GstMPDNode mediaPresentationDuration
GstMPDNode minimumUpdatePeriod
GstMPDNode minBufferTime
GstMPDNode timeShiftBufferDepth
GstMPDNode maxSegmentDuration
GstMPDNode maxSubsegmentDuration

(I'm not sure if GstMPDNode's suggestedPresentationDelay is allowed to have
negative values.)

These fields have type gint64 and are set to -1 if the information is not found
in the xml file. Ideally they should have the type GstClockTime and a default
value of GST_CLOCK_TIME_NONE if not present. But that require changes in the
API and careful changes in the code, because they currently represent values
measured in milliseconds and GstClockTime holds values measured in nanoseconds.

At least we should check that their values are positive and reject xml files
with negative values for those fields.

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