Page MenuHomeFreeBSD
Differential D58701

net: don't panic on ifconfig pfsync0 mtu 9000
ClosedPublic
Actions

Authored by kp on Aug 7 2026, 2:54 PM.
Tags
None
Referenced Files
F170808579: D58701.id183675.diff
Sun, Sep 6, 6:41 PM
F170722388: D58701.id183619.diff
Sun, Sep 6, 5:33 AM
F170697513: D58701.id183619.diff
Sun, Sep 6, 2:10 AM
Unknown Object (File)
Sat, Sep 5, 1:04 PM
Unknown Object (File)
Sat, Sep 5, 9:59 AM
Unknown Object (File)
Fri, Sep 4, 3:28 AM
Unknown Object (File)
Thu, Sep 3, 7:33 AM
Unknown Object (File)
Wed, Sep 2, 10:01 PM
View All 59 Files
Subscribers
ae
glebius
imp
melifaro
pouria

Details

Reviewers
glebius
Group Reviewers
network
pfsense
Commits
rG8ea4e92945d3: net: don't panic on ifconfig pfsync0 mtu 9000
Summary

pfsync interfaces do not have ifp->if_inet6 set, so when we update the
MTU for those interfaces we panicked.

Add an explicit check for this. This should be temporary, until pfsync
is no longer a struct ifnet (as we've already done for pflog).

Sponsored by: Rubicon Communications, LLC ("Netgate")

Diff Detail

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

Event Timeline

kp created this revision.Aug 7 2026, 2:54 PM
Herald added a reviewer: melifaro. · View Herald TranscriptAug 7 2026, 2:54 PM
Herald added subscribers: pouria, glebius, melifaro and 2 others. · View Herald Transcript
kp requested review of this revision.Aug 7 2026, 2:54 PM
Harbormaster completed remote builds in B75432: Diff 183619.Aug 7 2026, 2:54 PM
kp edited reviewers, added: glebius; removed: melifaro.Aug 7 2026, 2:55 PM
glebius accepted this revision as: glebius.Aug 7 2026, 9:23 PM
glebius added inline comments.
sys/net/route.c
519–521

Let's mark it with IFT_PFSYNC so that won't be forgotten to be removed.

This revision is now accepted and ready to land.Aug 7 2026, 9:23 PM
kp updated this revision to Diff 183674.Sat, Aug 8, 8:10 AM

Add IFT_PFSYNC comment.

This revision now requires review to proceed.Sat, Aug 8, 8:10 AM
Harbormaster completed remote builds in B75453: Diff 183674.Sat, Aug 8, 8:10 AM
This revision was not accepted when it landed; it landed in state Needs Review.Sat, Aug 8, 9:07 AM
Closed by commit rG8ea4e92945d3: net: don't panic on ifconfig pfsync0 mtu 9000 (authored by kp). · Explain Why
This revision was automatically updated to reflect the committed changes.
kp added a commit: rG8ea4e92945d3: net: don't panic on ifconfig pfsync0 mtu 9000.

Revision Contents
Changeset List

PathSize
sys/
net/
4 lines

Diff 183675

View Options

sys/net/route.c

Loading...