Differential D58854

net80211: fix WEP transmit
ClosedPublic
Actions

Authored by adrian on Aug 15 2026, 2:41 AM.

Tags

Referenced Files

	Unknown Object (File)
	Sun, Sep 27, 4:30 PM

	Unknown Object (File)
	Sun, Sep 27, 4:13 PM

	Unknown Object (File)
	Sun, Sep 27, 8:18 AM

	Unknown Object (File)
	Sun, Sep 27, 3:28 AM

	Unknown Object (File)
	Sun, Sep 27, 2:50 AM

	Unknown Object (File)
	Thu, Sep 24, 11:52 PM

	Unknown Object (File)
	Wed, Sep 23, 3:38 PM

	Unknown Object (File)
	Tue, Sep 22, 3:03 PM

View All 58 Files

Subscribers

Details

Reviewers

Commits

rG874035c11cd4: net80211: fix WEP transmit

Summary

This was broken in 2022 with a security fix (61605e0ae5d8f) which
disallowed defaulting to the default TX key if there's no unicast
key. Unfortunately this path was also used by WEP transmit.

To fix it, add a separate check which ensures that WEP is configured
(authtype OPEN, privacy enabled) - then also check if the default TX
key is set and that said key is a WEP key.

Fixes: 61605e0ae5d8f

Locally tested:

rtwn(4) AP and rtwn(4) STA w/ static WEP keys configured

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

adrian created this revision.Aug 15 2026, 2:41 AM

Herald added subscribers: glebius, melifaro, imp. · View Herald TranscriptAug 15 2026, 2:41 AM

adrian requested review of this revision.Aug 15 2026, 2:41 AM

Harbormaster completed remote builds in B75600: Diff 184084.Aug 15 2026, 2:41 AM

adrian added a parent revision: D58705: net80211: add key get/set methods.Aug 15 2026, 2:42 AM

adrian added a child revision: D58384: net80211: migrate the ioctl API to a 128 bit specific API + use key API.

adrian added a reviewer: bz.Aug 15 2026, 2:44 AM

adrian added a project: wireless.

also it would be nice to count/log the drops here...

This revision was not accepted when it landed; it landed in state Needs Review.Aug 19 2026, 2:07 AM

Closed by commit rG874035c11cd4: net80211: fix WEP transmit (authored by adrian). · Explain Why

This revision was automatically updated to reflect the committed changes.

adrian added a commit: rG874035c11cd4: net80211: fix WEP transmit.

Revision Contents
Changeset List

Path

Size

sys/

net80211/

ieee80211_crypto.c

18 lines

Diff 184299

sys/net80211/ieee80211_crypto.c

Loading...