Page MenuHomeFreeBSD
Differential D19005

ifconfig: fix endianness bug displaying pfsync interfaces
ClosedPublic
Actions

Authored by asomers on Jan 28 2019, 4:48 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Sep 6, 2:29 PM
Unknown Object (File)
Tue, Sep 2, 3:21 PM
Unknown Object (File)
Wed, Aug 27, 12:21 AM
Unknown Object (File)
Aug 7 2025, 11:07 AM
Unknown Object (File)
Aug 3 2025, 1:44 PM
Unknown Object (File)
Jul 12 2025, 10:03 PM
Unknown Object (File)
Jun 23 2025, 4:43 AM
Unknown Object (File)
Jun 22 2025, 4:28 AM
View All 100 Files
Subscribers
imp
kp

Details

Reviewers
glebius
kp
Commits
rS345397: MFC r343530, r344559
rS344138: MFC r343530:
rS343530: ifconfig: fix endianness bug displaying pfsync interfaces
Summary

ifconfig: fix endianness bug displaying pfsync interfaces

Test Plan

ifconfig tun0 create
kldload pfsync
ifconfig pfsync syncdev tun0
ifconfig pfsync0 # Verify that peer address doesn't display

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

asomers created this revision.Jan 28 2019, 4:48 PM
Herald added a subscriber: imp. · View Herald TranscriptJan 28 2019, 4:48 PM
Harbormaster completed remote builds in B22216: Diff 53316.Jan 28 2019, 4:48 PM
kp added a subscriber: kp.Jan 28 2019, 5:10 PM
kp added inline comments.
sbin/ifconfig/ifpfsync.c
198 ↗(On Diff #53316)

I think that needs to be 'htonl()'.

It'll work of course, but INADDR_PFSYNC_GROUP is in host byte order, and preq.pfsyncr_syncpeer.s_addr is in network order, so you want to go from host to network order, and not the other way around.

asomers added inline comments.Jan 28 2019, 5:23 PM
sbin/ifconfig/ifpfsync.c
198 ↗(On Diff #53316)

Oh, my mistake. I saw the definition and read it as a byte array, ignoring the leading (in_addr_t)

#define INADDR_PFSYNC_GROUP     ((in_addr_t)0xe00000f0) /* 224.0.0.240 */
asomers updated this revision to Diff 53324.Jan 28 2019, 5:24 PM

s/ntohl/htonl

Harbormaster completed remote builds in B22222: Diff 53324.Jan 28 2019, 5:24 PM
kp accepted this revision.Jan 28 2019, 5:44 PM
This revision is now accepted and ready to land.Jan 28 2019, 5:44 PM
Closed by commit rS343530: ifconfig: fix endianness bug displaying pfsync interfaces (authored by asomers). · Explain WhyJan 28 2019, 6:34 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
head/
sbin/
ifconfig/
6 lines

Diff 53336

View Options

head/sbin/ifconfig/ifpfsync.c

Loading...