[gstreamer-bugs] [Bug 632548] New: [rtspsrc] regression; fails to report duration

[GStreamer (bugzilla.gnome.org)]

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

           Summary: [rtspsrc] regression; fails to report duration
    Classification: Desktop
           Product: GStreamer
           Version: git
        OS/Version: Linux
            Status: UNCONFIRMED
          Severity: blocker
          Priority: Normal
         Component: gst-plugins-good
        AssignedTo: gstreamer-bugs at lists.sourceforge.net
        ReportedBy: mnauw at users.sourceforge.net
         QAContact: gstreamer-bugs at lists.sourceforge.net
      GNOME target: ---
     GNOME version: ---


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

rtspsrc: init segment prior to storing duration info in it

After some reshuffling in rtspsrc a while ago, current code order in
open_from_sdp is now (a.o.):

...
  /* parse range for duration reporting. */
  /* -> stored in src->segment */
...
  gst_segment_init (&src->segment, GST_FORMAT_TIME);
  /* ==> losing result obtaining above */

Since servers (apparently) might reply with a "npt=0-" range, duration info is
lost.

Provided patch restores earlier behaviour (i.e. retaining duration info
obtained from SDP).

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