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 724790 - dashdemux: live MPDs are immediately reloaded
dashdemux: live MPDs are immediately reloaded
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-20 10:40 UTC by Chris Bass
Modified: 2014-02-20 19:27 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
Initialise demux->last_manifest_update (959 bytes, patch)
2014-02-20 10:40 UTC, Chris Bass 		committed Details | Review

Description Chris Bass 2014-02-20 10:40:01 UTC 
Created attachment 269788 [details] [review]
Initialise demux->last_manifest_update

Due to demux->last_manifest_update not being initialised to GST_CLOCK_TIME_NONE, live MPDs with a minimumUpdatePeriod attribute are reloaded as soon as the download loop starts, i.e., the MPD is downloaded and parsed twice in quick succession.

Attached patch fixes this.
Comment 1 Sebastian Dröge (slomo) 2014-02-20 19:27:19 UTC 
commit 462c04cade43a03facb8b24bd9f1fdd19dd78244
Author: Chris Bass <floobleflam@gmail.com>
Date:   Tue Feb 18 10:19:52 2014 +0000

    dashdemux: initialise demux->last_manifest_update
    
    demux->last_manifest_update is not initialised at startup, with the
    effect that live manifests are reloaded immediately after the download
    loop begins. This patch fixes this.
    
    https://bugzilla.gnome.org/show_bug.cgi?id=724790