Page MenuHomeFreeBSD
Differential D5398

net80211: fix TIM cleanup
ClosedPublic
Actions

Authored by avos on Feb 22 2016, 11:39 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mar 20 2024, 9:06 PM
Unknown Object (File)
Feb 10 2024, 2:20 AM
Unknown Object (File)
Jan 2 2024, 9:22 PM
Unknown Object (File)
Dec 20 2023, 12:37 AM
Unknown Object (File)
Dec 18 2023, 8:35 PM
Unknown Object (File)
Sep 7 2023, 4:31 AM
Unknown Object (File)
Sep 7 2023, 4:29 AM
Unknown Object (File)
Sep 7 2023, 4:28 AM
View All 34 Files
Subscribers
imp

Details

Reviewers
adrian
Commits
rS295932: net80211: fix TIM cleanup.
Summary

Remove duplicate 'ni->ni_associd = 0' assignment from ieee80211_node_leave(), since it breaks iv_set_tim() in ic->ic_node_cleanup() (associd is cleared right after this call).

TODO: data races?

Test Plan

Tested with RTL8188EU (HOSTAP mode) and WUSB54GC (STA mode, with powersaving enabled).

Diff Detail

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

Event Timeline

avos updated this revision to Diff 13619.Feb 22 2016, 11:39 PM
avos retitled this revision from to net80211: fix TIM cleanup.
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.
Herald added a subscriber: imp. · View Herald TranscriptFeb 22 2016, 11:39 PM
adrian accepted this revision.Feb 23 2016, 5:53 AM
adrian edited edge metadata.

hah, good catch!

This revision is now accepted and ready to land.Feb 23 2016, 5:53 AM
Closed by commit rS295932: net80211: fix TIM cleanup. (authored by avos). · Explain WhyFeb 23 2016, 9:12 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
head/
sys/
net80211/
1 line

Diff 13661

View Options

head/sys/net80211/ieee80211_node.c

Loading...