Skip site navigation (1)Skip section navigation (2)
Date:      Mon, 19 Feb 2024 08:07:58 GMT
From:      "Bjoern A. Zeeb" <bz@FreeBSD.org>
To:        src-committers@FreeBSD.org, dev-commits-src-all@FreeBSD.org, dev-commits-src-branches@FreeBSD.org
Subject:   git: 5427cefd1d03 - stable/13 - net80211: fix checks for (*iv_preamble_update)/(*iv_ht_protmode_update)
Message-ID:  <202402190807.41J87wDr016239@gitrepo.freebsd.org>

next in thread | raw e-mail | index | archive | help
The branch stable/13 has been updated by bz:

URL: https://cgit.FreeBSD.org/src/commit/?id=5427cefd1d03e428a0148a6f65a39c280fa38374

commit 5427cefd1d03e428a0148a6f65a39c280fa38374
Author:     Bjoern A. Zeeb <bz@FreeBSD.org>
AuthorDate: 2024-01-29 20:59:05 +0000
Commit:     Bjoern A. Zeeb <bz@FreeBSD.org>
CommitDate: 2024-02-19 08:02:01 +0000

    net80211: fix checks for (*iv_preamble_update)/(*iv_ht_protmode_update)
    
    Both vap_update_preamble() and vap_update_ht_protmode() also check for
    (*iv_erp_protmode_update)() rather than (*iv_preamble_update)()
    or (*iv_ht_protmode_update)() before calling the later.
    Use the appropriate NULL-function-pointer checks before calling it.
    
    All seem unused currently so no functional changes expected.
    
    Fixes:          f1481c8d3b58e
    Reviewed by:    cc
    Differential Revision: https://reviews.freebsd.org/D43655
    
    (cherry picked from commit 48d689d6cabe41f9c04e75b774ef5b3e45b94469)
---
 sys/net80211/ieee80211_proto.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/sys/net80211/ieee80211_proto.c b/sys/net80211/ieee80211_proto.c
index 5ed9f2e3f50e..d00b7de0ab31 100644
--- a/sys/net80211/ieee80211_proto.c
+++ b/sys/net80211/ieee80211_proto.c
@@ -1051,7 +1051,7 @@ vap_update_preamble(void *arg, int npending)
 	IEEE80211_UNLOCK(ic);
 
 	/* Driver notification */
-	if (vap->iv_erp_protmode_update != NULL)
+	if (vap->iv_preamble_update != NULL)
 		vap->iv_preamble_update(vap);
 }
 
@@ -1191,7 +1191,7 @@ vap_update_ht_protmode(void *arg, int npending)
 	  ic->ic_curhtprotmode);
 
 	/* Driver update */
-	if (vap->iv_erp_protmode_update != NULL)
+	if (vap->iv_ht_protmode_update != NULL)
 		vap->iv_ht_protmode_update(vap);
 }
 



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?202402190807.41J87wDr016239>