mirror of
https://gitlab.freedesktop.org/NetworkManager/NetworkManager.git
synced 2026-02-28 11:00:49 +01:00
devices: fail optional-802.1X connections if supplicant disappears
802-1x.optional=yes means that NM should tolerate a failure or a timeout of the 802.1X authentication and should keep the connection up. Even if the authentication doesn't succeed, NM keeps the supplicant running so that it can continue trying. If the supplicant disappears because it crashed or was killed externally, NM should fail the connection so that it can be retried. The current code is wrong also because after releasing the supplicant interface, it calls wired_auth_cond_fail() which tries to connect a signal to priv->supplicant.iface (which is NULL). https://bugzilla.redhat.com/show_bug.cgi?id=1934291 https://gitlab.freedesktop.org/NetworkManager/NetworkManager/-/merge_requests/776 (cherry picked from commit840e54a96c) (cherry picked from commitb4ba6e7af5)
This commit is contained in:
parent
237e4e0545
commit
d3d5b807d6
1 changed files with 3 additions and 1 deletions
|
|
@ -726,7 +726,9 @@ supplicant_iface_state_cb(NMSupplicantInterface *iface,
|
|||
|
||||
if (new_state == NM_SUPPLICANT_INTERFACE_STATE_DOWN) {
|
||||
supplicant_interface_release(self);
|
||||
wired_auth_cond_fail(self, NM_DEVICE_STATE_REASON_SUPPLICANT_FAILED);
|
||||
nm_device_state_changed(NM_DEVICE(self),
|
||||
NM_DEVICE_STATE_FAILED,
|
||||
NM_DEVICE_STATE_REASON_SUPPLICANT_FAILED);
|
||||
return;
|
||||
}
|
||||
|
||||
|
|
|
|||
Loading…
Add table
Reference in a new issue