Basic support for ipfilter. It provides similar examples to ipfw and pf in the same file. See D6753 for the ipfw version of this diff.
Diff Detail
- Repository
- rS FreeBSD src repository - subversion
- Lint
Lint Skipped - Unit
Tests Skipped
Event Timeline
When you test this, make sure you test it with IPv6 traffic too.
blacklistd-helper | ||
---|---|---|
19 | I dislike that "ipfilter" uses "ipf.conf", but I guess that's just the way it is. | |
56 | This doesn't look right. It's going expand to something like this for ssh traffic: Maybe that last 'head $port$6' ought to be 'head port$6"... Also, I thought you had to flag IPv6 rules with '-6' to get them to be accepted. Both pf and ipfw will "just deal" with IPv6 addresses. | |
81 | I don't think this is right either. |
blacklistd-helper | ||
---|---|---|
19 | I suppose we could change it to ipfilter.conf but that would be a POLA violation. The customer can call it whatever he wants. | |
56 | There should be a space between = and port and $port$6 should be port $6. No -6 is required any more. ipf 5.1.2 no longer puts the rules in separate tables. |
This review can be abandoned. The ipfilter support was added several months ago. Any problems with the code would need a new review.