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 704255 - qtdemux: invalid base time on newsegment in push mode after a seek
qtdemux: invalid base time on newsegment in push mode after a seek
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gst-plugins-good
1.1.2
Other Linux
: Normal normal
: 1.1.3
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2013-07-15 14:31 UTC by Arnaud Vrac
Modified: 2013-07-16 08:48 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
qtdemux: reset segment on flush stop (1.55 KB, patch)
2013-07-15 14:31 UTC, Arnaud Vrac 		committed Details | Review

Description Arnaud Vrac 2013-07-15 14:31:39 UTC 
Created attachment 249205 [details] [review]
qtdemux: reset segment on flush stop

cca2f555d14 introduces a regression, where the demux segment is not reset on flush stop, so the next upstream segment event will calculate an invalid base time on the new segment to be sent downstream.

This can be reproduced easily by seeking in an mp4 file in push mode, after the seek it takes some time for playback to start again; this is because the segment base time is not 0, where it should be.
Comment 1 Sebastian Dröge (slomo) 2013-07-16 08:47:59 UTC 
commit 54bba4f60cfb6263c85f79167f0f5d7fa39fd548
Author: Arnaud Vrac <avrac@freebox.fr>
Date:   Mon Jul 15 16:24:07 2013 +0200

    qtdemux: reset segment on flush stop
    
    cca2f555d14 introduces a regression, where the demux segment is not
    reset on flush stop, so the next upstream segment event will calculate
    an invalid base time on the new segment to be sent downstream.
    
    https://bugzilla.gnome.org/show_bug.cgi?id=704255