mirrors/gstreamer
mirrors/gstreamer
1
1
Fork 0
mirror of https://gitlab.freedesktop.org/gstreamer/gstreamer.git synced 2024-11-27 04:01:08 +00:00
Code Issues 1 Projects Releases Wiki Activity

init: Fix initialisation crash

Browse source 
Fix a case where initialisation fails without setting
the passed-in GError and the caller assumes it will be
set, and add a guard to catch the condition in case it
happens again in the future.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gst-editing-services/-/merge_requests/217>
This commit is contained in:
Jan Schmidt 2020-10-30 00:30:52 +11:00
parent e267d06344
commit 180c23108b
2 changed files with 9 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

3
ges/ges.c
View file

@ -120,6 +120,9 @@ ges_init_post (GOptionContext * context, GOptionGroup * group, gpointer data,
_init_formatter_assets (); _init_formatter_assets ();
if (!_ges_uri_asset_ensure_setup (uriasset_klass)) { if (!_ges_uri_asset_ensure_setup (uriasset_klass)) {
GST_ERROR ("cannot setup uri asset"); GST_ERROR ("cannot setup uri asset");
if (error)
*error = g_error_new (GST_CORE_ERROR, GST_CORE_ERROR_MISSING_PLUGIN,
"Cannot initialize URI asset class.");
goto failed; goto failed;
} }

8
tools/ges-launcher.c
View file

@ -1051,9 +1051,13 @@ _local_command_line (GApplication * application, gchar ** arguments[],
gst_init (&argc, arguments); gst_init (&argc, arguments);
if (!ges_launcher_parse_options (self, arguments, &argc, ctx, &error)) { if (!ges_launcher_parse_options (self, arguments, &argc, ctx, &error)) {
gst_init (NULL, NULL); gst_init (NULL, NULL);
ges_printerr ("Error initializing: %s\n", error->message);
g_option_context_free (ctx); g_option_context_free (ctx);
g_error_free (error); if (error) {
ges_printerr ("Error initializing: %s\n", error->message);
g_error_free (error);
} else {
ges_printerr ("Error parsing command line arguments\n");
}
*exit_status = 1; *exit_status = 1;
goto done; goto done;
} }