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 723268 - hlsdemux: Does not take into account stop time of seek events
hlsdemux: Does not take into account stop time of seek events
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gst-plugins-bad
1.x
Other Linux
: Normal normal
: 1.3.1
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2014-01-30 09:07 UTC by Thibault Saunier
Modified: 2014-03-30 19:22 UTC
See Also:
GNOME target: ---
GNOME version: ---



Description Thibault Saunier 2014-01-30 09:07:28 UTC
The hlsdemux does not take into account end time of seek events.

You can easily reproduce using:

gst-validate-1.0  --set-scenario seek_forward playbin uri=http://devimages.apple.com/iphone/samples/bipbop/gear4/prog_index.m3u8

And will notice that the last seek has stop=30 * GST_SECOND but the video keeps running until the end.

No code implements that logic in hlsdemux.
Comment 1 Sebastian Dröge (slomo) 2014-02-04 12:32:36 UTC
And of course the decoders can't handle that either because they might be replaced since the seek event already. hlsdemux should keep the stop time in its segment that is sent to the downstream elements, but itself also stop at a fragment that starts after the stop time.
Comment 2 Sebastian Dröge (slomo) 2014-03-06 22:17:53 UTC
commit d2880dce68591d9b85ba777abcb4f4e4f6baa70c
Author: Sebastian Dröge <sebastian@centricular.com>
Date:   Thu Mar 6 23:14:10 2014 +0100

    hlsdemux: Implement proper segment handling
    
    https://bugzilla.gnome.org/show_bug.cgi?id=695846
    https://bugzilla.gnome.org/show_bug.cgi?id=723268
Comment 3 Sebastian Dröge (slomo) 2014-03-30 19:22:15 UTC
And a bit more

commit 89ca4535ebdf6272f7767934d66052d160327313
Author: Sebastian Dröge <sebastian@centricular.com>
Date:   Sun Mar 30 18:12:07 2014 +0200

    hlsdemux: Go EOS if the end of the segment is reached