Page MenuHomeFreeBSD

pf: Add a sysctl to limit work done for rdr source port rewriting
ClosedPublic

Authored by markj on Aug 30 2024, 5:30 PM.
Tags
None
Referenced Files
F102185428: D46495.id143149.diff
Fri, Nov 8, 4:06 PM
Unknown Object (File)
Wed, Nov 6, 11:27 AM
Unknown Object (File)
Wed, Oct 16, 7:14 AM
Unknown Object (File)
Tue, Oct 15, 4:02 PM
Unknown Object (File)
Mon, Oct 14, 12:37 AM
Unknown Object (File)
Sun, Oct 13, 6:52 AM
Unknown Object (File)
Sat, Oct 12, 10:01 AM
Unknown Object (File)
Fri, Oct 11, 2:34 AM

Details

Summary

It was pointed out that the current approach of exhaustively searching
for a free source port might be very time consuming. Limit the amount
of work that we might do before giving up.

Diff Detail

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

Event Timeline

markj requested review of this revision.Aug 30 2024, 5:30 PM

We should probably document this in pf(4).
And I should see about adding the others that are missing from that too.

This revision is now accepted and ready to land.Aug 30 2024, 7:58 PM

Mention the sysctl in pf.4 and pf.conf.5.

This revision now requires review to proceed.Sep 9 2024, 5:42 PM

I know I'm listed as a reviewer, but I can't assess the code change here. The gist of it looks good, though. I'd be a lot less worried about enabling the feature with this limiter in place.

It may want a man page date update.

This revision is now accepted and ready to land.Sep 9 2024, 5:55 PM