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)
Sat, Jul 25, 9:50 PM
Unknown Object (File)
Wed, Jul 22, 7:05 PM
Unknown Object (File)
Wed, Jul 22, 6:51 PM
Unknown Object (File)
Mon, Jul 20, 10:53 PM
Unknown Object (File)
Tue, Jul 14, 2:02 AM
Unknown Object (File)
Mon, Jul 13, 5:32 AM
Unknown Object (File)
Sat, Jul 4, 1:58 PM
Unknown Object (File)
Sat, Jul 4, 7:14 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

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

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
head/
sys/
net/
7 lines
9 lines
11 lines
9 lines
10 lines
8 lines
ofed/
drivers/
infiniband/
ulp/
ipoib/
9 lines

Diff 40818

View Options

head/sys/net/if_arcsubr.c

Loading...
View Options

head/sys/net/if_ethersubr.c

Loading...
View Options

head/sys/net/if_fddisubr.c

Loading...
View Options

head/sys/net/if_fwsubr.c

Loading...
View Options

head/sys/net/if_iso88025subr.c

Loading...
View Options

head/sys/net/if_vlan.c

Loading...
View Options

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

Loading...