Page MenuHomeFreeBSD
Differential D4043

net80211: eliminate copy-paste nearby ieee80211_check_rxseq()
ClosedPublic
Actions

Authored by avos on Oct 30 2015, 8:38 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Jan 20, 10:33 PM
Unknown Object (File)
Mon, Jan 20, 12:31 PM
Unknown Object (File)
Wed, Jan 15, 5:02 AM
Unknown Object (File)
Sat, Jan 11, 7:08 AM
Unknown Object (File)
Mon, Jan 6, 9:11 PM
Unknown Object (File)
Fri, Jan 3, 6:54 AM
Unknown Object (File)
Fri, Jan 3, 6:54 AM
Unknown Object (File)
Fri, Jan 3, 5:56 AM
View All 63 Files
Subscribers
imp

Details

Reviewers
adrian
Commits
rS296254: net80211: eliminate copy-paste nearby ieee80211_check_rxseq()

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

avos updated this revision to Diff 9835.Oct 30 2015, 8:38 PM
avos retitled this revision from to net80211: eliminate copy-paste nearby ieee80211_check_rxseq().
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 TranscriptOct 30 2015, 8:38 PM
adrian added inline comments.Oct 30 2015, 10:00 PM
sys/net80211/ieee80211_adhoc.c
441

.. what about the rxseqs[] assignment here? Where's it now being done?

avos added inline comments.Oct 30 2015, 10:16 PM
sys/net80211/ieee80211_adhoc.c
441

in the ieee80211_check_rxseq() ('ok' path)

adrian added inline comments.Oct 30 2015, 10:27 PM
sys/net80211/ieee80211_adhoc.c
441

It looks like in this patch you removed all of the instances though!

avos added inline comments.Oct 30 2015, 10:30 PM
sys/net80211/ieee80211_input.h
241

this?

avos added inline comments.Oct 31 2015, 10:59 PM
sys/net80211/ieee80211_adhoc.c
441

I've just moved duplicated code into ieee80211_check_rxseq() (so, this patch should be a noop)

adrian accepted this revision.Mar 1 2016, 4:29 AM
adrian edited edge metadata.

damn, how'd I miss that! sorry! go for it!

This revision is now accepted and ready to land.Mar 1 2016, 4:29 AM
Closed by commit rS296254: net80211: eliminate copy-paste nearby ieee80211_check_rxseq() (authored by avos). · Explain WhyMar 1 2016, 6:47 AM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
sys/
net80211/
19 lines
19 lines
26 lines
19 lines
19 lines
17 lines

Diff 9835

View Options

sys/net80211/ieee80211_adhoc.c

Loading...
View Options

sys/net80211/ieee80211_hostap.c

Loading...
View Options

sys/net80211/ieee80211_input.h

Loading...
View Options

sys/net80211/ieee80211_mesh.c

Loading...
View Options

sys/net80211/ieee80211_sta.c

Loading...
View Options

sys/net80211/ieee80211_wds.c

Loading...