summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorDan Williams <dcbw@redhat.com>2014-12-05 14:30:36 -0600
committerDan Williams <dcbw@redhat.com>2014-12-16 16:11:02 -0600
commitdc4d39a2d4582eb5092651f5a86f6e77488dafa0 (patch)
treea6711ca6ce8902afcd74659fc2ebbc9cb541fde8
parent8dd6a3b60075700ce2c0976670bd1d55fd5ae135 (diff)
downloadNetworkManager-dc4d39a2d4582eb5092651f5a86f6e77488dafa0.tar.gz
core: don't assume connections for INTERNAL or PARENT unmanaged devices
INTERNAL is actually a nop right now because the only thing that sets it is suspend/resume, which is covered by the preceding manager_sleeping() call. But we may use this more in the future, so add it while we're here. Devices that are unmanaged because their parent is unmanaged probably shouldn't assume connections either, per 4e105c50.
-rw-r--r--src/nm-manager.c4
1 files changed, 3 insertions, 1 deletions
diff --git a/src/nm-manager.c b/src/nm-manager.c
index 06f24abdb8..bed07e4162 100644
--- a/src/nm-manager.c
+++ b/src/nm-manager.c
@@ -1653,7 +1653,9 @@ recheck_assume_connection (NMDevice *device, gpointer user_data)
if (manager_sleeping (self))
return FALSE;
- if (nm_device_get_unmanaged_flag (device, NM_UNMANAGED_USER))
+ if (nm_device_get_unmanaged_flag (device, NM_UNMANAGED_USER) ||
+ nm_device_get_unmanaged_flag (device, NM_UNMANAGED_INTERNAL) ||
+ nm_device_get_unmanaged_flag (device, NM_UNMANAGED_PARENT))
return FALSE;
state = nm_device_get_state (device);