56 lines
2.5 KiB
Diff
56 lines
2.5 KiB
Diff
commit 912152cf85d29db45f706522c8e3ce13eaf13197
|
|
Author: Jiří Klimeš <jklimes@redhat.com>
|
|
Date: Tue Oct 29 15:02:30 2013 +0100
|
|
|
|
ifcfg-rh: fix crash when doing managed->unmanaged transition
|
|
|
|
Testcase:
|
|
* add 'NM_CONTROLLED=no' to /etc/sysconfig/network-scripts/ifcfg-ABC
|
|
* sudo nmcli con reload
|
|
* ... NM asserts ...
|
|
|
|
We need to ref() 'existing' connection before nm_settings_connection_signal_remove(),
|
|
because the function unref()s ithe connection via connection_removed_cb().
|
|
|
|
Backtrace:
|
|
...
|
|
#4 0x00007fbcf0ea0cba in g_assertion_message_expr (domain=domain@entry=0x0,
|
|
file=file@entry=0x7fbcf4e5805d "nm-dbus-manager.c", line=line@entry=848,
|
|
func=func@entry=0x7fbcf4e585e0 <__FUNCTION__.15088> "nm_dbus_manager_unregister_object", expr=expr@entry=0x7fbcf4e5820b "G_IS_OBJECT (object)")
|
|
at gtestutils.c:2293
|
|
#5 0x00007fbcf4de69d9 in nm_dbus_manager_unregister_object (
|
|
self=0x7fbcf6fdc9c0, object=0x7fbcf70235c0) at nm-dbus-manager.c:848
|
|
#6 0x00007fbcf4dd6a23 in nm_settings_connection_signal_remove (
|
|
self=<optimized out>) at settings/nm-settings-connection.c:1541
|
|
#7 0x00007fbce6fee884 in connection_new_or_changed (
|
|
self=self@entry=0x7fbcf7006f80,
|
|
path=path@entry=0x7fbcf70c3f80 "/etc/sysconfig/network-scripts/ifcfg-ABC",
|
|
existing=existing@entry=0x7fbcf70235c0,
|
|
out_old_path=out_old_path@entry=0x7fff2b7b8988) at plugin.c:327
|
|
#8 0x00007fbce6feeca2 in read_connections (plugin=0x7fbcf7006f80)
|
|
at plugin.c:453
|
|
#9 0x00007fbcf4dd8e98 in impl_settings_reload_connections (
|
|
self=0x7fbcf6fd98c0, context=0x7fbcf70bcb30) at settings/nm-settings.c:1262
|
|
...
|
|
|
|
diff --git a/src/settings/plugins/ifcfg-rh/plugin.c b/src/settings/plugins/ifcfg-rh/plugin.c
|
|
index dbfc496..5cd24b0 100644
|
|
--- a/src/settings/plugins/ifcfg-rh/plugin.c
|
|
+++ b/src/settings/plugins/ifcfg-rh/plugin.c
|
|
@@ -320,6 +320,7 @@ connection_new_or_changed (SCPluginIfcfg *self,
|
|
|
|
if (new_unmanaged) {
|
|
if (!old_unmanaged) {
|
|
+ g_object_ref (existing);
|
|
/* Unexport the connection by telling the settings service it's
|
|
* been removed, and notify the settings service by signalling that
|
|
* unmanaged specs have changed.
|
|
@@ -331,6 +332,7 @@ connection_new_or_changed (SCPluginIfcfg *self,
|
|
|
|
g_object_set (existing, NM_IFCFG_CONNECTION_UNMANAGED, new_unmanaged, NULL);
|
|
g_signal_emit_by_name (self, NM_SYSTEM_CONFIG_INTERFACE_UNMANAGED_SPECS_CHANGED);
|
|
+ g_object_unref (existing);
|
|
}
|
|
} else {
|
|
if (old_unmanaged) { /* now managed */
|