Differential D58854

net80211: fix WEP transmit
ClosedPublic
Actions

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

Tags

Referenced Files

	Unknown Object (File)
	Mon, Sep 7, 5:56 AM

	Unknown Object (File)
	Sun, Sep 6, 11:34 PM

	Unknown Object (File)
	Sun, Sep 6, 6:16 PM

	Unknown Object (File)
	Sat, Sep 5, 8:05 PM

	Unknown Object (File)
	Fri, Sep 4, 12:43 PM

	Unknown Object (File)
	Wed, Sep 2, 11:20 PM

	Unknown Object (File)
	Wed, Sep 2, 11:17 PM

	Unknown Object (File)
	Tue, Sep 1, 6:35 PM

View All 39 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.Sat, Aug 15, 2:41 AM

Herald added subscribers: glebius, melifaro, imp. · View Herald TranscriptSat, Aug 15, 2:41 AM

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

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

adrian added a parent revision: D58705: net80211: add key get/set methods.Sat, Aug 15, 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.Sat, Aug 15, 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.Wed, Aug 19, 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...