Page MenuHomeFreeBSD

net80211: improve error checking in ieee80211_parse_{wpa,rsn}
ClosedPublic

Authored by avos on Sep 13 2016, 9:38 PM.
Tags
None
Referenced Files
F162903748: D7887.diff
Sat, Jul 18, 4:02 AM
Unknown Object (File)
Tue, Jul 14, 2:51 PM
Unknown Object (File)
Mon, Jul 6, 8:28 AM
Unknown Object (File)
Sat, Jul 4, 10:15 AM
Unknown Object (File)
Sat, Jul 4, 6:16 AM
Unknown Object (File)
Fri, Jul 3, 8:23 AM
Unknown Object (File)
Thu, Jul 2, 7:34 AM
Unknown Object (File)
Jun 10 2026, 8:48 AM
Subscribers

Details

Summary
  • Add few checks for group/pairwise ciphers in ieee80211_parse_{wpa,rsn}.
  • Split error code and cipher value in wpa_cipher() / rsn_cipher(); current hack with (1 << 32) does not work - it's 1, not 0 (originally detected by CSA).
  • Return IEEE80211_REASON_UNSUPP_RSN_IE_VERSION instead of IEEE80211_REASON_IE_INVALID when version field is not equal to RSN_VERSION.
Test Plan

Tested with wpi(4) / urtwn(4); no visible regressions.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

avos retitled this revision from to net80211: improve error checking in ieee80211_parse_{wpa,rsn}.
avos updated this object.
avos edited the test plan for this revision. (Show Details)
avos added a reviewer: adrian.
avos set the repository for this revision to rS FreeBSD src repository - subversion.
adrian edited edge metadata.

looks good to me!

This revision is now accepted and ready to land.Sep 13 2016, 10:02 PM
This revision was automatically updated to reflect the committed changes.