diff options
author | Sebastian Dröge <sebastian.droege@collabora.co.uk> | 2010-08-08 11:56:42 +0200 |
---|---|---|
committer | Sebastian Dröge <sebastian.droege@collabora.co.uk> | 2010-08-08 11:56:42 +0200 |
commit | 1e3ec9e3bd798c7d9c30e328702a293ff53ec344 (patch) | |
tree | ab58c24c46b10ad2207de67f25096fa70e3a70b0 | |
parent | e2571b9f46038a962a306aa71df11e1e21161834 (diff) |
frei0r: Don't try to register/load the same frei0r plugin at different locations twice
This could happen because for example /usr/lib is linked
to /usr/lib64 and both are loaded. The frei0r specification
says that the plugin init function must only be called once
and for some plugin weird things (including crashes) are
happening.
Fixes bug #623710.
-rw-r--r-- | gst/frei0r/gstfrei0r.c | 38 |
1 files changed, 27 insertions, 11 deletions
diff --git a/gst/frei0r/gstfrei0r.c b/gst/frei0r/gstfrei0r.c index 6d2c6fce4..b78343591 100644 --- a/gst/frei0r/gstfrei0r.c +++ b/gst/frei0r/gstfrei0r.c @@ -588,12 +588,13 @@ invalid_frei0r_plugin: } static gboolean -register_plugins (GstPlugin * plugin, const gchar * path) +register_plugins (GstPlugin * plugin, GHashTable * plugin_names, + const gchar * path) { GDir *dir; gchar *filename; const gchar *entry_name; - gboolean ret = FALSE; + gboolean ret = FALSE, this_ret; GST_DEBUG ("Scanning director '%s' for frei0r plugins", path); @@ -602,16 +603,24 @@ register_plugins (GstPlugin * plugin, const gchar * path) return FALSE; while ((entry_name = g_dir_read_name (dir))) { + this_ret = FALSE; + + if (g_hash_table_lookup_extended (plugin_names, entry_name, NULL, NULL)) + continue; + filename = g_build_filename (path, entry_name, NULL); if ((g_str_has_suffix (filename, G_MODULE_SUFFIX) #ifdef GST_EXTRA_MODULE_SUFFIX || g_str_has_suffix (filename, GST_EXTRA_MODULE_SUFFIX) #endif ) && g_file_test (filename, G_FILE_TEST_IS_REGULAR)) { - ret |= register_plugin (plugin, filename); + this_ret = register_plugin (plugin, filename); } else if (g_file_test (filename, G_FILE_TEST_IS_DIR)) { - ret |= register_plugins (plugin, filename); + this_ret = register_plugins (plugin, plugin_names, filename); } + if (this_ret) + g_hash_table_insert (plugin_names, g_strdup (entry_name), NULL); + ret = ret && this_ret; g_free (filename); } g_dir_close (dir); @@ -624,6 +633,7 @@ plugin_init (GstPlugin * plugin) { const gchar *homedir; gchar *path; + GHashTable *plugin_names; GST_DEBUG_CATEGORY_INIT (frei0r_debug, "frei0r", 0, "frei0r"); @@ -636,17 +646,23 @@ plugin_init (GstPlugin * plugin) "/usr/lib64/frei0r-1:/usr/local/lib64/frei0r-1", NULL, GST_PLUGIN_DEPENDENCY_FLAG_RECURSE); + plugin_names = + g_hash_table_new_full ((GHashFunc) g_str_hash, (GEqualFunc) g_str_equal, + (GDestroyNotify) g_free, NULL); + homedir = g_get_home_dir (); path = g_build_filename (homedir, ".frei0r-1", NULL); - register_plugins (plugin, path); + register_plugins (plugin, plugin_names, path); g_free (path); - register_plugins (plugin, "/usr/local/lib/frei0r-1"); - register_plugins (plugin, "/usr/lib/frei0r-1"); - register_plugins (plugin, "/usr/local/lib32/frei0r-1"); - register_plugins (plugin, "/usr/lib32/frei0r-1"); - register_plugins (plugin, "/usr/local/lib64/frei0r-1"); - register_plugins (plugin, "/usr/lib64/frei0r-1"); + register_plugins (plugin, plugin_names, "/usr/local/lib/frei0r-1"); + register_plugins (plugin, plugin_names, "/usr/lib/frei0r-1"); + register_plugins (plugin, plugin_names, "/usr/local/lib32/frei0r-1"); + register_plugins (plugin, plugin_names, "/usr/lib32/frei0r-1"); + register_plugins (plugin, plugin_names, "/usr/local/lib64/frei0r-1"); + register_plugins (plugin, plugin_names, "/usr/lib64/frei0r-1"); + + g_hash_table_unref (plugin_names); return TRUE; } |