Page MenuHomeFreeBSD
Differential D28655

ifconfig-wg fix incorrect allowed-ips netmask info
ClosedPublic
Actions

Authored by nyan_myuji.xyz on Feb 13 2021, 11:39 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, May 24, 8:16 AM
Unknown Object (File)
Thu, May 23, 12:58 PM
Unknown Object (File)
Mon, May 20, 4:48 PM
Unknown Object (File)
Mon, May 20, 9:25 AM
Unknown Object (File)
Mon, May 20, 7:57 AM
Unknown Object (File)
Thu, May 16, 11:30 PM
Unknown Object (File)
Tue, May 7, 10:55 AM
Unknown Object (File)
Tue, May 7, 1:58 AM
View All 46 Files
Subscribers
imp
khng
Contributor Reviews (src)

Details

Reviewers
grehan
mmacy
Group Reviewers
network
Commits
rGf28545524cb1: ifconfig: fix incorrect wg allowed-ips netmask
rG0646b83f0779: ifconfig: fix incorrect wg allowed-ips netmask
rGbe2003ccfa1b: ifconfig: fix incorrect wg allowed-ips netmask
Summary

Currently when peer information being dumped via ifconfig wgN peer ..
or ifconfig wgN peer-list, whichever the netmask of the first allowed-ips
will be used as the netmask of all CIDR in allowed-ips. For example, if the
list is 192.168.1.0/24, 172.16.0.0/16, it will display as
192.168.1.0/24, 172.16.0.0/24. While this does not affects the actual functionality,
it creates massive confusion for users.

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

nyan_myuji.xyz created this revision.Feb 13 2021, 11:39 PM
Herald added subscribers: Contributor Reviews (src), imp. · View Herald TranscriptFeb 13 2021, 11:39 PM
nyan_myuji.xyz requested review of this revision.Feb 13 2021, 11:39 PM
Harbormaster completed remote builds in B36979: Diff 83853.Feb 13 2021, 11:39 PM
nyan_myuji.xyz added a comment.Feb 13 2021, 11:42 PM

Related bugzillla ticket: https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=253487

kp added reviewers: grehan, mmacy, network.Feb 14 2021, 8:14 AM
grehan accepted this revision.Feb 14 2021, 8:34 AM
This revision is now accepted and ready to land.Feb 14 2021, 8:34 AM
khng added a subscriber: khng.Feb 14 2021, 8:59 AM

This works for me.

Closed by commit rGbe2003ccfa1b: ifconfig: fix incorrect wg allowed-ips netmask (authored by nyan_myuji.xyz, committed by philip). · Explain WhyFeb 15 2021, 2:40 AM
This revision was automatically updated to reflect the committed changes.
philip added a commit: rGbe2003ccfa1b: ifconfig: fix incorrect wg allowed-ips netmask.
philip added a commit: rG0646b83f0779: ifconfig: fix incorrect wg allowed-ips netmask.Feb 16 2021, 2:12 AM
philip added a commit: rGf28545524cb1: ifconfig: fix incorrect wg allowed-ips netmask.Feb 16 2021, 9:14 PM

Revision Contents
Changeset List

PathSize
sbin/
ifconfig/
2 lines

Diff 83903

View Options

sbin/ifconfig/ifwg.c

Loading...