pfctl - Change default keep-policy to bring more in-line with other BSDs
authorMatthew Dillon <dillon@apollo.backplane.com>
Sat, 28 Jun 2014 02:05:33 +0000 (19:05 -0700)
committerMatthew Dillon <dillon@apollo.backplane.com>
Sat, 28 Jun 2014 02:05:33 +0000 (19:05 -0700)
commitf72d5ff0d8dd6b9afab31a2e1d51d5539bf85b03
tree375bf58553c742e4a4cc118e69407ca8fb4980d7
parent32e6d213123a90463d92ffe47f4afa150d4af50a
pfctl - Change default keep-policy to bring more in-line with other BSDs

* Change the default keep-policy to the equivalent of:

  set keep-policy keep state (pickups, sloppy)

* This is being done because without keep state PF is simply going to be
  too inefficient for any reasonable set of rules, and we no longer want
  to make users set the keep-policy line when keep state is already the
  default in other BSD systems.

* Note that we also set pickups and sloppy by default.  This allows the
  router and/or PF to be restarted and allows packet routing to change
  mid-stream without causing all active TCP connections to drop.  This
  may not be the default in other BSD systems but it should be.  Being
  ultra strict here to improve security against ICMP-based attacks removes
  too much flexibility to be appropriate.  Proper TCP implementations
  already do sequence space checks for RST packets.
usr.sbin/pfctl/parse.y