about summary refs log tree commit diff
path: root/src/devices/ovs
diff options
context:
space:
mode:
authorMichael Biebl <biebl@debian.org>2020-03-13 00:45:02 +0100
committerMichael Biebl <biebl@debian.org>2020-03-13 00:45:02 +0100
commitdae4785d3a29c5df3740c2676b400c0da1a72e3e (patch)
tree9fe17185cbbcbd096c85e5973e1b909ae3f751e4 /src/devices/ovs
parent3cbb3d6ecfef1076fc757632ee8327935250afee (diff)
parentb012fa6e1d808e0736c009799c62d835cbfcc1dd (diff)
Update upstream source from tag 'upstream/1.22.10'
Update to upstream version '1.22.10'
with Debian dir df10a9f9ccf26e17902b3dc19a866a22c47764e8
Diffstat (limited to 'src/devices/ovs')
-rw-r--r--src/devices/ovs/nm-device-ovs-port.c13
1 files changed, 11 insertions, 2 deletions
diff --git a/src/devices/ovs/nm-device-ovs-port.c b/src/devices/ovs/nm-device-ovs-port.c
index 0955e8a9..e0216377 100644
--- a/src/devices/ovs/nm-device-ovs-port.c
+++ b/src/devices/ovs/nm-device-ovs-port.c
@@ -86,6 +86,7 @@ add_iface_cb (GError *error, gpointer user_data)
 static gboolean
 enslave_slave (NMDevice *device, NMDevice *slave, NMConnection *connection, gboolean configure)
 {
+	NMDeviceOvsPort *self = NM_DEVICE_OVS_PORT (device);
 	NMActiveConnection *ac_port = NULL;
 	NMActiveConnection *ac_bridge = NULL;
 	NMDevice *bridge_device;
@@ -95,10 +96,18 @@ enslave_slave (NMDevice *device, NMDevice *slave, NMConnection *connection, gboo
 
 	ac_port = NM_ACTIVE_CONNECTION (nm_device_get_act_request (device));
 	ac_bridge = nm_active_connection_get_master (ac_port);
-	if (!ac_bridge)
-		ac_bridge = ac_port;
+	if (!ac_bridge) {
+		_LOGW (LOGD_DEVICE, "can't enslave %s: bridge active-connection not found",
+		       nm_device_get_iface (slave));
+		return FALSE;
+	}
 
 	bridge_device = nm_active_connection_get_device (ac_bridge);
+	if (!bridge_device) {
+		_LOGW (LOGD_DEVICE, "can't enslave %s: bridge device not found",
+		       nm_device_get_iface (slave));
+		return FALSE;
+	}
 
 	nm_ovsdb_add_interface (nm_ovsdb_get (),
 	                        nm_active_connection_get_applied_connection (ac_bridge),