diff options
author | Kristof Provost <kp@FreeBSD.org> | 2023-10-13 10:34:06 +0000 |
---|---|---|
committer | Kristof Provost <kp@FreeBSD.org> | 2023-10-15 14:21:53 +0000 |
commit | 6ace92326f3cc4bbee9b74f6b656cb13699a78a2 (patch) | |
tree | 271c501901cc422d7533ff3533d6c90d78bd3c84 /sbin | |
parent | f34c9c4e3bdc2b8bffae4ac26897e0e847e9f76f (diff) | |
download | src-6ace92326f3cc4bbee9b74f6b656cb13699a78a2.tar.gz src-6ace92326f3cc4bbee9b74f6b656cb13699a78a2.zip |
ifconfig: add static bridge addresses for vlan 0, not 1
Since fd7edfcdc3 ("bridge: fix lookup for untagged packets in
bridge_transmit()") and b0e38a1373 ("bridge: distinguish no vlan and
vlan 1") we do a better job of distinguishing between untagged and VLAN
1 traffic.
However, ifconfig still defaulted to adding addresses for VLAN 1, rather
than for untagged traffic. Change this to be the most common (i.e.
untagged) option.
Reviewed by: zlei, philip
MFC after: 3 days
Differential Revision: https://reviews.freebsd.org/D42188
Diffstat (limited to 'sbin')
-rw-r--r-- | sbin/ifconfig/ifbridge.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/sbin/ifconfig/ifbridge.c b/sbin/ifconfig/ifbridge.c index 9b499b404946..2d0af1255a73 100644 --- a/sbin/ifconfig/ifbridge.c +++ b/sbin/ifconfig/ifbridge.c @@ -402,7 +402,7 @@ setbridge_static(if_ctx *ctx, const char *val, const char *mac) memcpy(req.ifba_dst, ea->octet, sizeof(req.ifba_dst)); req.ifba_flags = IFBAF_STATIC; - req.ifba_vlan = 1; /* XXX allow user to specify */ + req.ifba_vlan = 0; /* XXX allow user to specify */ if (do_cmd(ctx, BRDGSADDR, &req, sizeof(req), 1) < 0) err(1, "BRDGSADDR %s", val); |