Page MenuHomeFreeBSD
Differential D49148

wlanwds: convert getifaddrs() iteration to use the ifa_next pointer
ClosedPublic
Actions

Authored by adrian on Feb 27 2025, 2:25 AM.
Tags
Referenced Files
Unknown Object (File)
Sun, Aug 16, 9:52 AM
Unknown Object (File)
Thu, Aug 13, 12:21 AM
Unknown Object (File)
Tue, Aug 11, 6:12 PM
Unknown Object (File)
Mon, Aug 10, 11:49 AM
Unknown Object (File)
Mon, Aug 10, 8:51 AM
Unknown Object (File)
Sat, Aug 8, 5:21 AM
Unknown Object (File)
Fri, Aug 7, 5:37 PM
Unknown Object (File)
Wed, Aug 5, 9:04 AM
View All Files
Subscribers
imp

Details

Reviewers
None
Group Reviewers
wireless
Commits
rG36347cfbf7ac: wlanwds: convert getifaddrs() iteration to use the ifa_next pointer
Summary

I don't know what I was thinking in 2017 when I wrote this, but
it's very wrong.

Use ifa->ifa_next to iterate, not the array index / pointer arithmetic.

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Skipped
Unit
Tests Skipped
Build Status
Buildable 62648
Build 59532: arc lint + arc unit

Event Timeline

adrian created this revision.Feb 27 2025, 2:25 AM
Herald added a subscriber: imp. · View Herald TranscriptFeb 27 2025, 2:25 AM
adrian requested review of this revision.Feb 27 2025, 2:25 AM
Harbormaster completed remote builds in B62648: Diff 151554.Feb 27 2025, 2:25 AM
adrian updated this revision to Diff 151555.Feb 27 2025, 2:25 AM

update

Harbormaster completed remote builds in B62649: Diff 151555.Feb 27 2025, 2:25 AM
adrian added a child revision: D48996: wlanwds: use a local copy of the ifaddr sockaddr, avoid cast.Feb 27 2025, 2:27 AM
adrian mentioned this in D48996: wlanwds: use a local copy of the ifaddr sockaddr, avoid cast.
adrian added a reviewer: wireless.
adrian added a project: wireless.
adrian added a child revision: D49149: wlanwatch: clean up to compile / run.Feb 27 2025, 2:39 AM
adrian removed a child revision: D49149: wlanwatch: clean up to compile / run.
This revision was not accepted when it landed; it landed in state Needs Review.Feb 28 2025, 2:51 AM
Closed by commit rG36347cfbf7ac: wlanwds: convert getifaddrs() iteration to use the ifa_next pointer (authored by adrian). · Explain Why
This revision was automatically updated to reflect the committed changes.
adrian added a commit: rG36347cfbf7ac: wlanwds: convert getifaddrs() iteration to use the ifa_next pointer.

Revision Contents
Changeset List

PathSize
tools/
tools/
net80211/
wlanwds/
2 lines

Diff 151554

View Options

tools/tools/net80211/wlanwds/wlanwds.c

Loading...