ifconfig/ifbridge.c: add get_vlan_id()
ClosedPublic
Actions

Authored by ivy on Jul 25 2025, 11:45 PM.

Details

Reviewers

des
kp

Commits

rG8b728c63f777: ifconfig/ifbridge.c: add get_vlan_id()
rG287a5fdcd3c9: ifconfig/ifbridge.c: add get_vlan_id()

Summary

This is like get_val() but takes an ether_vlanid_t* and ensures the
value is a valid VLAN ID. This avoids redundant comparisons and
casting when parsing VLAN IDs.

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Skipped

Unit

Tests Skipped

Build Status

Buildable 65745
Build 62628: arc lint + arc unit

Event Timeline

ivy created this revision.Jul 25 2025, 11:45 PM

Herald added a subscriber: imp. · View Herald TranscriptJul 25 2025, 11:45 PM

ivy requested review of this revision.Jul 25 2025, 11:45 PM

Harbormaster completed remote builds in B65745: Diff 159164.Jul 25 2025, 11:45 PM

standalone version of D51244 in an attempt to land these bridge changes before 15.0.

des accepted this revision.Jul 28 2025, 2:08 PM

This revision is now accepted and ready to land.Jul 28 2025, 2:08 PM

Closed by commit rG287a5fdcd3c9: ifconfig/ifbridge.c: add get_vlan_id() (authored by ivy). · Explain WhyJul 28 2025, 2:10 PM

This revision was automatically updated to reflect the committed changes.

ivy added a commit: rG287a5fdcd3c9: ifconfig/ifbridge.c: add get_vlan_id().

ivy added a commit: rG8b728c63f777: ifconfig/ifbridge.c: add get_vlan_id().Aug 23 2025, 12:12 AM

Revision Contents
Changeset List

Path

Size

sbin/

ifconfig/

ifbridge.c

28 lines

Diff 159164

View Options

ifconfig/ifbridge.c: add get_vlan_id()ClosedPublicActions