GNOME Bugzilla – Bug 671132
hlsdemux: live streams stops prematurely
Last modified: 2016-12-30 09:35:35 UTC
Created attachment 208787 [details] [review] patch to fix issue When I try to play a live HLS stream the playback usually stops with an EOS after fetching the first few segments. This is caused by a buggy implementation of the live index reload code which erroneously reports index updates. I have attached a patch that will fix the implementation.
Do you by any chance have a sample URI at hand where that happens?
(In reply to comment #1) > Do you by any chance have a sample URI at hand where that happens? The only URL I have is geo-blocked to danish ip's: http://lm.gss.dr.dk/V/V01H.stream/Playlist.m3u8
If you contact me on IRC (nick kanongil) I can provide a non geo-blocked URL.
Created attachment 216918 [details] [review] hlsdemux: don't copy data from past the end of the buffer
Created attachment 216919 [details] [review] hlsdemux: also skip the newline character
Created attachment 216920 [details] [review] hlsdemux: actually preserve the input data The current implementation fails to ever detect unchanged input data, since it compares it to a modified version, rather than the original.
Created attachment 216921 [details] [review] hlsdemux: make gst_m3u8 manage update_failed_count
I have split my patch into several smaller patches and updated it to apply to the current master.
Comment on attachment 216918 [details] [review] hlsdemux: don't copy data from past the end of the buffer Looks like this was fixed independently (if only people would check bugzilla for patches before they go on a bug fixing spree ;)): commit 80a3a014f566b828df9ad68b4ee9925d3dc103ec Author: Sebastian Dröge <sebastian.droege@collabora.co.uk> Date: Fri Oct 12 15:09:07 2012 +0200 hlsdemux: Fix invalid read
Is this still a problem with latest GIT or 1.2.2? Does someone have a stream to reproduce?
Yes, still problem with 1.2.3 Try http://46.249.55.30:80/first_1200_00.m3u8?user=zixi_test for test
The whole hlsdemux codebase has been rewritten. Is this still an issue?
Closing this bug report as no further information has been provided. Please feel free to reopen this bug report if you can provide the information that was asked for in a previous comment. Thanks!