datapath: Avoid assigning a NULL pointer to flow actions.
authorJarno Rajahalme <jrajahalme@nicira.com>
Tue, 25 Mar 2014 00:34:48 +0000 (17:34 -0700)
committerJarno Rajahalme <jrajahalme@nicira.com>
Tue, 25 Mar 2014 00:34:48 +0000 (17:34 -0700)
commit90b8c2f7c08eab4aea41601e57d4eee021ef5561
tree3959414a304d7af5e41b25ecdeaed854ab78ef6c
parent87968f01fc8404b98f0bb98aac0f73b743b79b5c
datapath: Avoid assigning a NULL pointer to flow actions.

Flow SET can accept an empty set of actions, with the intended
semantics of leaving existing actions unmodified.  This seems to have
been brokin after OVS 1.7, as we have assigned the flow's actions
pointer to NULL in this case, but we never check for the NULL pointer
later on.  This patch restores the intended behavior and documents it
in the include/linux/openvswitch.h.

Signed-off-by: Jarno Rajahalme <jrajahalme@nicira.com>
Signed-off-by: Pravin B Shelar <pshelar@nicira.com>
datapath/datapath.c
include/linux/openvswitch.h