From 726f830c107fe57a05accd173658d56b4788ed19 Mon Sep 17 00:00:00 2001 From: Thomas Haller Date: Fri, 29 Apr 2016 15:00:09 +0200 Subject: device: don't unload settings plugins on failure Also, registering a weak-pointer to close the module that was just made as resident is pointless. --- src/settings/nm-settings.c | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/src/settings/nm-settings.c b/src/settings/nm-settings.c index b91247a9ae..da0a59493e 100644 --- a/src/settings/nm-settings.c +++ b/src/settings/nm-settings.c @@ -847,18 +847,19 @@ load_plugin: break; } + /* after accessing the plugin we cannot unload it anymore, because the glib + * types cannot be properly unregistered. */ + g_module_make_resident (plugin); + obj = (*factory_func) (); if (!obj || !NM_IS_SETTINGS_PLUGIN (obj)) { g_set_error (error, NM_SETTINGS_ERROR, NM_SETTINGS_ERROR_FAILED, "Plugin '%s' returned invalid system config object.", pname); success = FALSE; - g_module_close (plugin); break; } - g_module_make_resident (plugin); - g_object_weak_ref (obj, (GWeakNotify) g_module_close, plugin); g_object_set_data_full (obj, PLUGIN_MODULE_PATH, path, g_free); path = NULL; if (add_plugin (self, NM_SETTINGS_PLUGIN (obj))) -- cgit v1.2.1