xml-formatter: Add an GST_ERROR when setting control sources fails
authorThibault Saunier <tsaunier@igalia.com>
Wed, 27 May 2020 23:39:49 +0000 (19:39 -0400)
committerGStreamer Merge Bot <gitlab-merge-bot@gstreamer-foundation.org>
Wed, 27 May 2020 23:49:00 +0000 (23:49 +0000)
CID 1463853

Part-of: <https://gitlab.freedesktop.org/gstreamer/gst-editing-services/-/merge_requests/180>

ges/ges-base-xml-formatter.c

index d9149cc..7239ba6 100644 (file)
@@ -1115,8 +1115,11 @@ ges_base_xml_formatter_add_control_binding (GESBaseXmlFormatter * self,
         property_name, binding_type);
 
     g_object_set (source, "mode", mode, NULL);
-    gst_timed_value_control_source_set_from_list (GST_TIMED_VALUE_CONTROL_SOURCE
-        (source), timed_values);
+    if (!gst_timed_value_control_source_set_from_list
+        (GST_TIMED_VALUE_CONTROL_SOURCE (source), timed_values)) {
+      GST_ERROR_OBJECT (self, "Could not set timed values on %" GES_FORMAT,
+          GES_ARGS (source));
+    }
 
     gst_object_unref (source);
   } else