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)
Mon, Jan 19, 10:09 AM
Unknown Object (File)
Fri, Jan 9, 6:24 AM
Unknown Object (File)
Wed, Jan 7, 12:31 PM
Unknown Object (File)
Tue, Jan 6, 11:15 AM
Unknown Object (File)
Sun, Jan 4, 11:50 AM
Unknown Object (File)
Wed, Dec 31, 2:04 AM
Unknown Object (File)
Wed, Dec 31, 2:04 AM
Unknown Object (File)
Wed, Dec 31, 2:04 AM
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...