Page MenuHomeFreeBSD
Differential D18779

Don't set if_linkmib for vlan(4).
ClosedPublic
Actions

Authored by markj on Jan 8 2019, 1:13 AM.
Tags
None
Referenced Files
F109028461: D18779.id52698.diff
Thu, Jan 30, 8:18 PM
F108988118: D18779.diff
Thu, Jan 30, 8:18 AM
F108972809: D18779.diff
Thu, Jan 30, 4:04 AM
Unknown Object (File)
Thu, Jan 23, 6:57 PM
Unknown Object (File)
Sat, Jan 18, 5:15 PM
Unknown Object (File)
Dec 14 2024, 8:15 PM
Unknown Object (File)
Oct 24 2024, 5:01 PM
Unknown Object (File)
Oct 24 2024, 4:37 PM
View All 66 Files
Subscribers
ae
imp
kp

Details

Reviewers
kp
Group Reviewers
network
Commits
rS342887: Stop setting if_linkmib in vlan(4) ifnets.
Summary

It doesn't really make sense:

  • the structure being exported isn't a MIB,
  • the structure being exported is private to the kernel and always has been, and
  • no other drivers in widespread use implement a link-specific MIB.

Moreover, the IFDATA_LINKSPECIFIC interface allows a privileged user to
overwrite a bunch of internal state without any validation.

Diff Detail

Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 21877
Build 21123: arc lint + arc unit

Event Timeline

markj created this revision.Jan 8 2019, 1:13 AM
Herald added a subscriber: ae. · View Herald TranscriptJan 8 2019, 1:13 AM
Harbormaster completed remote builds in B21877: Diff 52655.Jan 8 2019, 1:13 AM
markj added a reviewer: network.Jan 8 2019, 1:13 AM
kp accepted this revision as: kp.Jan 8 2019, 8:14 AM
kp added a subscriber: kp.

LGTM.

This revision is now accepted and ready to land.Jan 8 2019, 8:14 AM
markj updated this revision to Diff 52672.Jan 8 2019, 5:02 PM
  • Also collapse ifv_linkmib into the parent structure.
This revision now requires review to proceed.Jan 8 2019, 5:02 PM
Harbormaster completed remote builds in B21886: Diff 52672.Jan 8 2019, 5:02 PM
Harbormaster completed remote builds in B21886: Diff 52672.
This revision was not accepted when it landed; it landed in state Needs Review.Jan 9 2019, 4:47 PM
Closed by commit rS342887: Stop setting if_linkmib in vlan(4) ifnets. (authored by markj). · Explain Why
This revision was automatically updated to reflect the committed changes.
Herald added a subscriber: imp. · View Herald TranscriptJan 9 2019, 4:47 PM

Revision Contents
Changeset List

PathSize
sys/
net/
4 lines

Diff 52655

View Options

sys/net/if_vlan.c

Loading...