From 78bbaeb6e7adbb20a4bc6efe2057dd44999063a7 Mon Sep 17 00:00:00 2001 From: Thibault Saunier Date: Sat, 16 May 2015 11:17:40 +0200 Subject: [PATCH] timedvaluecontrolsource: Fix removing all keyframes, and adding one back We were segfaulting because g_sequence_search was returning the iter_end, and that iterator does not contain anything and thus should not be used directly --- libs/gst/controller/gsttimedvaluecontrolsource.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libs/gst/controller/gsttimedvaluecontrolsource.c b/libs/gst/controller/gsttimedvaluecontrolsource.c index c4fc6102d1..eb956ffa51 100644 --- a/libs/gst/controller/gsttimedvaluecontrolsource.c +++ b/libs/gst/controller/gsttimedvaluecontrolsource.c @@ -182,7 +182,7 @@ gst_timed_value_control_source_set_internal (GstTimedValueControlSource * iter = g_sequence_search (self->values, ×tamp, (GCompareDataFunc) gst_control_point_find, NULL); - if (iter) { + if (iter && !g_sequence_iter_is_end (iter)) { GSequenceIter *prev = g_sequence_iter_prev (iter); GstControlPoint *cp = g_sequence_get (prev);