git: 27bf5c405bf2 - main - net80211: Fix IEEE80211_KEY_UNDEFINED() in ieee80211_node_delucastkey()
- Go to: [ bottom of page ] [ top of archives ] [ this month ]
Date: Wed, 05 Mar 2025 21:19:00 UTC
The branch main has been updated by bz:
URL: https://cgit.FreeBSD.org/src/commit/?id=27bf5c405bf2eb69392e45c06605defc78882612
commit 27bf5c405bf2eb69392e45c06605defc78882612
Author: Bjoern A. Zeeb <bz@FreeBSD.org>
AuthorDate: 2025-03-05 00:02:50 +0000
Commit: Bjoern A. Zeeb <bz@FreeBSD.org>
CommitDate: 2025-03-05 21:18:15 +0000
net80211: Fix IEEE80211_KEY_UNDEFINED() in ieee80211_node_delucastkey()
The adjustment of the check in 9763fec11b83 inverted the logic. Fix this.
Sponsored by: The FreeBSD Foundation
MFC after: 3 days
Fixes: 9763fec11b83
Reported by: KUROSAWA Takahiro (takahiro.kurosawa gmail.com) in D48980
Reviewed by: KUROSAWA Takahiro (takahiro.kurosawa gmail.com), adrian
Differential Revision: https://reviews.freebsd.org/D49234
---
sys/net80211/ieee80211_node.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/sys/net80211/ieee80211_node.c b/sys/net80211/ieee80211_node.c
index f4185dfb04d4..a495f2f78113 100644
--- a/sys/net80211/ieee80211_node.c
+++ b/sys/net80211/ieee80211_node.c
@@ -2206,7 +2206,7 @@ ieee80211_node_delucastkey(struct ieee80211_node *ni)
IEEE80211_NODE_LOCK(nt);
nikey = NULL;
status = 1; /* NB: success */
- if (IEEE80211_KEY_UNDEFINED(&ni->ni_ucastkey)) {
+ if (!IEEE80211_KEY_UNDEFINED(&ni->ni_ucastkey)) {
keyix = ni->ni_ucastkey.wk_rxkeyix;
status = ieee80211_crypto_delkey(ni->ni_vap, &ni->ni_ucastkey);
if (nt->nt_keyixmap != NULL && keyix < nt->nt_keyixmax) {