Page MenuHomeFreeBSD

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

Authored by markj on Jan 8 2019, 1:13 AM.

Details

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

Repository
rS FreeBSD src repository
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

markj created this revision.Jan 8 2019, 1:13 AM
kristof accepted this revision as: kristof.Jan 8 2019, 8:14 AM
kristof added a subscriber: kristof.

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.
This revision was not accepted when it landed; it landed in state Needs Review.Jan 9 2019, 4:47 PM
This revision was automatically updated to reflect the committed changes.