From 8d445ee75836d6c4cd2706f294fb0d6e52c2a41d Mon Sep 17 00:00:00 2001 From: Luis de Bethencourt Date: Tue, 9 Feb 2016 21:35:19 +0000 Subject: [PATCH] mssdemux: remove unnecessary check stream->current_fragment has the value of g_list_previous (iter) which has just been checked. No need to check it again. Just to be safe, use a g_assert() to check fragment before dereferencing. CID #1352041 --- ext/smoothstreaming/gstmssmanifest.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/ext/smoothstreaming/gstmssmanifest.c b/ext/smoothstreaming/gstmssmanifest.c index 73c623f968..02e559cf9c 100644 --- a/ext/smoothstreaming/gstmssmanifest.c +++ b/ext/smoothstreaming/gstmssmanifest.c @@ -1149,8 +1149,8 @@ gst_mss_stream_seek (GstMssStream * stream, gboolean forward, } else if (stream->fragment_repetition_index == -1) { if (g_list_previous (iter)) { stream->current_fragment = g_list_previous (iter); - fragment = - stream->current_fragment ? stream->current_fragment->data : NULL; + fragment = stream->current_fragment->data; + g_assert (fragment); stream->fragment_repetition_index = fragment->repetitions - 1; } else { stream->fragment_repetition_index = 0;