Page MenuHomeFreeBSD
Differential D14873

Improve copy-and-paste versions of SIOCGIFADDR.
ClosedPublic
Actions

Authored by brooks on Mar 27 2018, 6:48 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, May 17, 11:03 PM
Unknown Object (File)
Fri, May 15, 5:44 AM
Unknown Object (File)
Thu, May 7, 9:33 AM
Unknown Object (File)
Mon, Apr 27, 2:35 PM
Unknown Object (File)
Sat, Apr 25, 6:49 AM
Unknown Object (File)
Thu, Apr 23, 12:41 PM
Unknown Object (File)
Apr 17 2026, 11:52 AM
Unknown Object (File)
Apr 11 2026, 7:56 PM
View All Files
Subscribers
ae
imp

Details

Reviewers
kib
emaste
jhb
Commits
rS332160: MFC r331648:
rS332159: MFC r331648:
rS331648: Improve copy-and-pasted versions of SIOCGIFADDR.
Summary

The original implementation used a reference to ifr_data and a cast to
do the equivalent of accessing ifr_addr. This was copied multiple
times since 1996.

Diff Detail

Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 15834
Build 15842: arc lint + arc unit

Event Timeline

brooks created this revision.Mar 27 2018, 6:48 PM
Herald added a subscriber: ae. · View Herald TranscriptMar 27 2018, 6:48 PM
Harbormaster completed remote builds in B15834: Diff 40814.Mar 27 2018, 6:48 PM
brooks added reviewers: kib, emaste, jhb.Mar 27 2018, 6:49 PM

This is the first of several changes to clean up use of ifr_data before placing all remaining access behind an accessor function.

kib accepted this revision.Mar 27 2018, 8:17 PM
This revision is now accepted and ready to land.Mar 27 2018, 8:17 PM
Closed by commit rS331648: Improve copy-and-pasted versions of SIOCGIFADDR. (authored by brooks). · Explain WhyMar 27 2018, 8:52 PM
This revision was automatically updated to reflect the committed changes.
Herald added a subscriber: imp. · View Herald TranscriptMar 27 2018, 8:52 PM

Revision Contents
Changeset List

PathSize
sys/
net/
7 lines
9 lines
11 lines
9 lines
10 lines
8 lines
ofed/
drivers/
infiniband/
ulp/
ipoib/
9 lines

Diff 40814

View Options

sys/net/if_arcsubr.c

Loading...
View Options

sys/net/if_ethersubr.c

Loading...
View Options

sys/net/if_fddisubr.c

Loading...
View Options

sys/net/if_fwsubr.c

Loading...
View Options

sys/net/if_iso88025subr.c

Loading...
View Options

sys/net/if_vlan.c

Loading...
View Options

sys/ofed/drivers/infiniband/ulp/ipoib/ipoib_main.c

Loading...