avformat/hls: fix segment selection regression on track changes of live streams
authorAnssi Hannula <anssi.hannula@iki.fi>
Thu, 15 Oct 2015 10:42:38 +0000 (13:42 +0300)
committerAnssi Hannula <anssi.hannula@iki.fi>
Thu, 15 Oct 2015 12:04:00 +0000 (15:04 +0300)
Commit ad701326b43078b90 ("avformat/hls: open playlists immediately when
AVDISCARD_ALL is dropped") inadvertently caused first_packet to never be
cleared, causing select_cur_seq_no() to not use the specific code for
live streams.

In practice this means that when the user selects a different audio
track during live stream (i.e. non-VOD) playback, there may be some
additional delay as the code might select an incorrect segment at first,
and we have to wait for video to catch audio (if too late segment was
selected) or to download more following audio segments (if too early
segment was selected).

Fix that by restoring the zeroing of first_packet.

libavformat/hls.c

index ff95519..3c073e0 100644 (file)
@@ -1602,6 +1602,7 @@ static int hls_read_packet(AVFormatContext *s, AVPacket *pkt)
     int ret, i, minplaylist = -1;
 
     recheck_discard_flags(s, c->first_packet);
+    c->first_packet = 0;
 
     for (i = 0; i < c->n_playlists; i++) {
         struct playlist *pls = c->playlists[i];