Differential D50787

net80211: fix VHT node setup in hostap mode
ClosedPublic
Actions

Authored by adrian on Jun 11 2025, 5:06 AM.

Tags

Referenced Files

	Unknown Object (File)
	Wed, Jul 16, 3:24 PM

	Unknown Object (File)
	Mon, Jul 7, 1:02 AM

	Unknown Object (File)
	Sat, Jul 5, 10:11 PM

	Unknown Object (File)
	Fri, Jul 4, 5:36 PM

	Unknown Object (File)
	Thu, Jul 3, 5:44 PM

	Unknown Object (File)
	Fri, Jun 27, 5:11 PM

	Unknown Object (File)
	Fri, Jun 20, 4:09 PM

	Unknown Object (File)
	Jun 15 2025, 11:25 AM

View All 20 Files

Subscribers

Details

Reviewers

None

Group Reviewers

Commits

rG4e78b9b5afe9: net80211: fix VHT node setup in hostap mode

Summary

Way back in 2017 I wrote the initial VHT hostap support, and then
added a NULL check here to stop it from panicing. However, there
is no VHTINFO IE in an ASSOC_REQ management frame. Thus, the node
would never have VHT state initialised.

This mirrors what HT node setup does (which again doesn't have
HTINFO IE's in an ASSOC_REQ frame), the upgrade will happen once
the exchange completes.

Fixes: 2023566223a4020d832bd4e25dc7dadfd660b836

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

adrian created this revision.Jun 11 2025, 5:06 AM

Herald added subscribers: glebius, melifaro, imp. · View Herald TranscriptJun 11 2025, 5:06 AM

adrian requested review of this revision.Jun 11 2025, 5:06 AM

Harbormaster completed remote builds in B64756: Diff 156809.Jun 11 2025, 5:06 AM

adrian added a parent revision: D50772: net80211: convert the rest of the native net80211 drivers to SEQNO_OFFLOAD.Jun 11 2025, 5:07 AM

adrian added a reviewer: wireless.

adrian added a project: wireless.

adrian added a child revision: D50794: net80211: don't dereference a NULL HTINFO IE if it's presented.Jun 11 2025, 6:19 PM

This revision was not accepted when it landed; it landed in state Needs Review.Wed, Jul 16, 3:22 PM

Closed by commit rG4e78b9b5afe9: net80211: fix VHT node setup in hostap mode (authored by adrian). · Explain Why

This revision was automatically updated to reflect the committed changes.

adrian added a commit: rG4e78b9b5afe9: net80211: fix VHT node setup in hostap mode.

Revision Contents
Changeset List

Path

Size

sys/

net80211/

ieee80211_hostap.c

7 lines

ieee80211_vht.h

3 lines

ieee80211_vht.c

4 lines

Diff 158612

sys/net80211/ieee80211_hostap.c

Loading...

sys/net80211/ieee80211_vht.h

Loading...

sys/net80211/ieee80211_vht.c

Loading...