]> CyberLeo.Net >> Repos - FreeBSD/stable/9.git/commit
Merge 232685 from head:
authorglebius <glebius@ccf9f872-aa2e-dd11-9fc8-001c23d0bc1f>
Wed, 21 Mar 2012 09:19:23 +0000 (09:19 +0000)
committerglebius <glebius@ccf9f872-aa2e-dd11-9fc8-001c23d0bc1f>
Wed, 21 Mar 2012 09:19:23 +0000 (09:19 +0000)
commita26d71172583b8b2f4d9f2a36384cf08bb195929
tree62ffc91b1c75bd82cbb1c0c94f37abc027237b22
parentb9acc3ef616a42f7adf1b3a07e7b955bbdc8b4a6
Merge 232685 from head:
  Merge from OpenBSD:

    revision 1.146
    date: 2010/05/12 08:11:11;  author: claudio;  state: Exp;  lines: +2 -3
    bzero() the full compressed update struct before setting the values.
    This is needed because pf_state_peer_hton() skips some fields in certain
    situations which could result in garbage beeing sent to the other peer.
    This seems to fix the pfsync storms seen by stephan@ and so dlg owes me
    a whiskey.

  I didn't see any storms, but this definitely fixes a useless memory
  allocation on the receiving side, due to non zero scrub_flags field
  in a pfsync_state_peer structure.

git-svn-id: svn://svn.freebsd.org/base/stable/9@233275 ccf9f872-aa2e-dd11-9fc8-001c23d0bc1f
sys/contrib/pf/net/if_pfsync.c