adaptivedemux: Error out if downloading a segment failed too often in live mode

It's true that we shouldn't consider errors fatal immediately, but if we
always ignore them we will loop infinitely on live streams with segments
that can't be downloaded at all.
This commit is contained in:
Sebastian Dröge 2015-06-05 11:26:11 +02:00
parent adcf3f48a5
commit 4dd355cea2

View file

@ -1896,9 +1896,11 @@ gst_adaptive_demux_stream_download_fragment (GstAdaptiveDemuxStream * stream)
if (ret != GST_FLOW_OK) { if (ret != GST_FLOW_OK) {
/* TODO check if we are truly stoping */ /* TODO check if we are truly stoping */
if (ret != GST_FLOW_ERROR && gst_adaptive_demux_is_live (demux)) { if (ret != GST_FLOW_ERROR && gst_adaptive_demux_is_live (demux)) {
/* looks like there is no way of knowing when a live stream has ended if (++stream->download_error_count <= MAX_DOWNLOAD_ERROR_COUNT) {
* Have to assume we are falling behind and cause a manifest reload */ /* looks like there is no way of knowing when a live stream has ended
return GST_FLOW_EOS; * Have to assume we are falling behind and cause a manifest reload */
return GST_FLOW_EOS;
}
} }
} }
} }