From a2dce28fc8f8319ab517f0bd4d5813f0a2acf98f Mon Sep 17 00:00:00 2001 From: Thomas Haller Date: Mon, 2 Jan 2017 15:47:16 +0100 Subject: [PATCH] device: do nothing in NMDevice's link_changed() function All implementations of link_changed() chain up to NMDevice's base implementation. Thus, everybody wants to set the carrier. Refactor the code to set the carrier outside of link_changed(). --- src/devices/nm-device.c | 13 +++++++------ .../plugins/ifcfg-rh/nms-ifcfg-rh-connection.c | 4 ++-- 2 files changed, 9 insertions(+), 8 deletions(-) diff --git a/src/devices/nm-device.c b/src/devices/nm-device.c index 8d1fd2331..8bfc9688c 100644 --- a/src/devices/nm-device.c +++ b/src/devices/nm-device.c @@ -1981,8 +1981,12 @@ device_link_changed (NMDevice *self) _LOGD (LOGD_DEVICE, "IPv6 tokenized identifier present on device %s", priv->iface); } - if (klass->link_changed) - klass->link_changed (self, &info); + /* Update carrier from link event if applicable. */ + if ( nm_device_has_capability (self, NM_DEVICE_CAP_CARRIER_DETECT) + && !nm_device_has_capability (self, NM_DEVICE_CAP_NONSTANDARD_CARRIER)) + nm_device_set_carrier (self, pllink->connected); + + klass->link_changed (self, &info); /* Update DHCP, etc, if needed */ if (ip_ifname_changed) @@ -2121,10 +2125,7 @@ link_changed_cb (NMPlatform *platform, static void link_changed (NMDevice *self, const NMPlatformLink *pllink) { - /* Update carrier from link event if applicable. */ - if ( nm_device_has_capability (self, NM_DEVICE_CAP_CARRIER_DETECT) - && !nm_device_has_capability (self, NM_DEVICE_CAP_NONSTANDARD_CARRIER)) - nm_device_set_carrier (self, pllink->connected); + /* stub implementation of virtual function to allow subclasses to chain up. */ } static gboolean diff --git a/src/settings/plugins/ifcfg-rh/nms-ifcfg-rh-connection.c b/src/settings/plugins/ifcfg-rh/nms-ifcfg-rh-connection.c index 92593644d..33f4847ed 100644 --- a/src/settings/plugins/ifcfg-rh/nms-ifcfg-rh-connection.c +++ b/src/settings/plugins/ifcfg-rh/nms-ifcfg-rh-connection.c @@ -197,8 +197,8 @@ nm_ifcfg_connection_check_devtimeout (NMIfcfgConnection *self) devtimeout, ifname, nm_connection_get_id (NM_CONNECTION (self))); priv->devtimeout_link_changed_handler = - g_signal_connect (NM_PLATFORM_GET, NM_PLATFORM_SIGNAL_LINK_CHANGED, - G_CALLBACK (link_changed), self); + g_signal_connect (NM_PLATFORM_GET, NM_PLATFORM_SIGNAL_LINK_CHANGED, + G_CALLBACK (link_changed), self); priv->devtimeout_timeout_id = g_timeout_add_seconds (devtimeout, devtimeout_expired, self); }