GNOME Bugzilla – Bug 726239
[review] jk/dev-disconnect: fix a regression in manual device disconnection
Last modified: 2014-03-13 15:08:39 UTC
Devices disconnected explicitly by user should stay disconnected, preventing auto-connecting until manual request. Introduction of NM_DEVICE_STATE_DEACTIVATING state broke this feature. disconnect_cb() correctly set autoconnect device property to FALSE, however nm_device_state_changed() put it to TRUE again. Thus only the active connection was blocked instead of the whole device.
A fix has been pushed to branch jk/dev-disconnect.
It looks like the switch() doesn't handle the FAILED case either. The intent of the autoconnect=TRUE reset is that if the device is activating or connected, it gets reset to TRUE. So we could either: 1) move the priv->autoconnect=TRUE into a separate if (state >= PREPARE && state <= ACTIVATED) priv->autoconnect=TRUE 2) add a FAILED case to the same block as DEACTIVATING
Looks good now (with option #1 implemented)
Pushed to master as: b02353e core: fix a regression in manual device disconnection (bgo #726239)