openvswitch: fix conntrack netlink event delivery
authorSamuel Gauthier <samuel.gauthier@6wind.com>
Tue, 28 Jun 2016 15:22:26 +0000 (17:22 +0200)
committerDavid S. Miller <davem@davemloft.net>
Wed, 29 Jun 2016 12:13:59 +0000 (08:13 -0400)
commitd913d3a763a6f66a862a6eafcf6da89a7905832a
tree5c64c9ad187b5116a7b69a3a9d4e82ed620047d5
parent34c7bb4705a0a2d344b0c82eaf3d3bfa2bc9da45
openvswitch: fix conntrack netlink event delivery

Only the first and last netlink message for a particular conntrack are
actually sent. The first message is sent through nf_conntrack_confirm when
the conntrack is committed. The last one is sent when the conntrack is
destroyed on timeout. The other conntrack state change messages are not
advertised.

When the conntrack subsystem is used from netfilter, nf_conntrack_confirm
is called for each packet, from the postrouting hook, which in turn calls
nf_ct_deliver_cached_events to send the state change netlink messages.

This commit fixes the problem by calling nf_ct_deliver_cached_events in the
non-commit case as well.

Fixes: 7f8a436eaa2c ("openvswitch: Add conntrack action")
CC: Joe Stringer <joestringer@nicira.com>
CC: Justin Pettit <jpettit@nicira.com>
CC: Andy Zhou <azhou@nicira.com>
CC: Thomas Graf <tgraf@suug.ch>
Signed-off-by: Samuel Gauthier <samuel.gauthier@6wind.com>
Acked-by: Joe Stringer <joe@ovn.org>
Signed-off-by: David S. Miller <davem@davemloft.net>
net/openvswitch/conntrack.c