public inbox for iwd@lists.linux.dev
 help / color / mirror / Atom feed
* [PATCH] netdev: destroy auth-proto after external auth
@ 2024-12-19 18:17 James Prestwood
  2024-12-19 18:18 ` James Prestwood
  2024-12-20  5:57 ` Denis Kenzior
  0 siblings, 2 replies; 5+ messages in thread
From: James Prestwood @ 2024-12-19 18:17 UTC (permalink / raw)
  To: iwd; +Cc: James Prestwood

With external auth there is no associate event meaning the auth proto
never gets freed, which prevents eapol from starting inside the
OCI callback. Check for this specific case and free the auth proto
after signaling that external auth has completed.
---
 src/netdev.c | 7 +++++++
 1 file changed, 7 insertions(+)

diff --git a/src/netdev.c b/src/netdev.c
index 02496c92..2a6d94fc 100644
--- a/src/netdev.c
+++ b/src/netdev.c
@@ -3483,6 +3483,13 @@ static void netdev_external_auth_sae_tx_associate(void *user_data)
 
 	netdev_send_external_auth(netdev, MMPDU_STATUS_CODE_SUCCESS);
 	netdev_ensure_eapol_registered(netdev);
+
+	/*
+	 * Free the auth proto now. With external auth there is no associate
+	 * event which is where this normally gets cleaned up.
+	 */
+	auth_proto_free(netdev->ap);
+	netdev->ap = NULL;
 }
 
 struct rtnl_data {
-- 
2.34.1


^ permalink raw reply related	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2024-12-20  5:57 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-12-19 18:17 [PATCH] netdev: destroy auth-proto after external auth James Prestwood
2024-12-19 18:18 ` James Prestwood
2024-12-20  3:57   ` KeithG
2024-12-20  4:58     ` KeithG
2024-12-20  5:57 ` Denis Kenzior

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox