After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 724875 - dashdemux: attempts to fetch fragments before they're available.
dashdemux: attempts to fetch fragments before they're available.
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gst-plugins-bad
git master
Other Linux
: Normal normal
: 1.3.1
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2014-02-21 12:40 UTC by Chris Bass
Modified: 2014-02-22 15:29 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
Fix incorrect time scaling (1.07 KB, patch)
2014-02-21 12:40 UTC, Chris Bass 		committed Details | Review

Description Chris Bass 2014-02-21 12:40:14 UTC 
Created attachment 269903 [details] [review]
Fix incorrect time scaling

In gst_dash_demux_wait_for_fragment_to_be_available, the time difference until the next fragment is available is converted to milliseconds before being passed to gst_dash_demux_download_wait. However, the latter expects a GstClockTime, and so the code will wait for a much shorter time than required and will therefore try to fetch the next segment before it's available.

Attached patch fixes this.
Comment 1 Sebastian Dröge (slomo) 2014-02-22 15:29:27 UTC 
commit 6342d225bff8371fce9452cbe1ae966ddec4041a
Author: Chris Bass <floobleflam@gmail.com>
Date:   Fri Feb 21 12:33:42 2014 +0000

    dashdemux: ensure segments aren't fetched before they're available
    
    Incorrect time scaling in gst_dash_demux_wait_for_fragment_to_be_available()
    means that media segments are fetched before their availablity time. This
    patch fixes this.
    
    https://bugzilla.gnome.org/show_bug.cgi?id=724875