summaryrefslogtreecommitdiff
path: root/svc/lib/cs
diff options
context:
space:
mode:
authorCalvin Morrison <calvin@pobox.com>2026-08-18 21:11:11 -0400
committerCalvin Morrison <calvin@pobox.com>2026-08-18 21:11:11 -0400
commit35f13d1f583d9e9b8e00011c625444ce6b5f1d9f (patch)
treed7383926292b3d893f99e03e1f1db66f7a43eb52 /svc/lib/cs
parent24ae4160d6b82aa409ec8f2a80d4b59166c19972 (diff)
doc: say what fragments do, and stop listing fixed items as open
todo.md had three items in both halves of itself. Control files being world-writable, delete 0, and a dead relay leaving half a firewall were all in "Fixed since the review" as items 10, 11 and 12, and all three were still sitting in "Worth fixing" underneath. Anyone working the open list would have redone them. Now an item appears in one half or the other, and the first round's entries are one line each, since the detail is in the commit and the value of this file is the part that is still true. Fragments got a sentence that described the mechanism and not the consequence. "Later fragments now match on addresses and protocol only" reads as though they would pass under an address rule; what actually happens is that every rule set in fwrules(6) is written in ports, later fragments match no port, and the datagram does not cross. That is the right trade against reading ports out of them, which let a crafted fragment through, but it is a hole in what works and belongs in BUGS rather than in a changelog line. man/fw also now says what the served /net contains, which changed underneath it and was never written down; and it still said the card was "not undone reliably", when nothing undoes it at all. design.md gains the whitelist decision next to the others, because the argument for it is the same one the rule parser already makes and the next person to add a file to the served tree should meet it. The mtu comment claimed a card as its reason while sitting in code both modes use. Between two stacks there is no card; 1500 is still right, for a reason worth one sentence. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Diffstat (limited to 'svc/lib/cs')
0 files changed, 0 insertions, 0 deletions