Page MenuHomeFreeBSD
Differential D53068

ipfw: Check for errors from sooptcopyin() and sooptcopyout()
ClosedPublic
Actions

Authored by markj on Oct 13 2025, 1:25 PM.
Tags
None
Referenced Files
F163142476: D53068.id164091.diff
Mon, Jul 20, 11:48 AM
F163135578: D53068.id.diff
Mon, Jul 20, 10:05 AM
F163106201: D53068.id164091.diff
Mon, Jul 20, 3:23 AM
Unknown Object (File)
Sat, Jul 18, 9:49 PM
Unknown Object (File)
Sat, Jul 18, 3:17 AM
Unknown Object (File)
Fri, Jul 17, 7:51 PM
Unknown Object (File)
Wed, Jul 15, 6:49 AM
Unknown Object (File)
Wed, Jul 15, 6:49 AM
View All 95 Files
Subscribers
donner
glebius
imp
melifaro
vegeta_tuxpowered.net

Details

Reviewers
ae
Commits
rGeaafcf841ce8: ipfw: Check for errors from sooptcopyin() and sooptcopyout()
rGc839eecec5e2: ipfw: Check for errors from sooptcopyin() and sooptcopyout()
rG6dfca589ea75: ipfw: Check for errors from sooptcopyin() and sooptcopyout()
rG302c4c540e44: ipfw: Check for errors from sooptcopyin() and sooptcopyout()
rG2df39ce5d4a8: ipfw: Check for errors from sooptcopyin() and sooptcopyout()
Summary

Note, it looks like this code may be unused since commit 4a77657cbc01
("ipfw: migrate ipfw to 32-bit size rule numbers"). In particular, it
looks like the ipfw_nat_*_ptr pointers are unused now.

Diff Detail

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

Event Timeline

markj created this revision.Oct 13 2025, 1:25 PM
Herald added subscribers: vegeta_tuxpowered.net, glebius, donner and 2 others. · View Herald TranscriptOct 13 2025, 1:25 PM
markj requested review of this revision.Oct 13 2025, 1:25 PM
Harbormaster completed remote builds in B67751: Diff 164091.Oct 13 2025, 1:25 PM
ae accepted this revision.Oct 14 2025, 7:38 AM

Yes, I think they were replaced with handlers from ipfw_sopt_handler.

This revision is now accepted and ready to land.Oct 14 2025, 7:38 AM
markj added a comment.Oct 14 2025, 1:15 PM
In D53068#1212662, @ae wrote:

Yes, I think they were replaced with handlers from ipfw_sopt_handler.

Ok. I will commit anyway since the change needs to be MFCed. Will you clean up the old handlers?

Closed by commit rG2df39ce5d4a8: ipfw: Check for errors from sooptcopyin() and sooptcopyout() (authored by markj). · Explain WhyOct 14 2025, 1:51 PM
This revision was automatically updated to reflect the committed changes.
markj added a commit: rG2df39ce5d4a8: ipfw: Check for errors from sooptcopyin() and sooptcopyout().
markj added a commit: rG302c4c540e44: ipfw: Check for errors from sooptcopyin() and sooptcopyout().Oct 22 2025, 12:52 PM
markj added a commit: rG6dfca589ea75: ipfw: Check for errors from sooptcopyin() and sooptcopyout().Oct 22 2025, 1:01 PM
markj added a commit: rGc839eecec5e2: ipfw: Check for errors from sooptcopyin() and sooptcopyout().
cperciva added a commit: rGeaafcf841ce8: ipfw: Check for errors from sooptcopyin() and sooptcopyout().Oct 22 2025, 8:15 PM

Revision Contents
Changeset List

PathSize
sys/
netpfil/
ipfw/
12 lines

Diff 164142

View Options

sys/netpfil/ipfw/ip_fw_nat.c

Loading...