openvswitch: Avoid assigning a NULL pointer to flow actions.
authorJarno Rajahalme <jrajahalme@nicira.com>
Mon, 5 May 2014 16:59:40 +0000 (09:59 -0700)
committerPravin B Shelar <pshelar@nicira.com>
Thu, 22 May 2014 23:27:34 +0000 (16:27 -0700)
commitbe52c9e96a6657d117bb0ec6e11438fb246af5c7
tree92764c69b9c3cda7a6263dd5163caeea57a6e815
parent1139e241ec436b9e9610c7a33ac5c6657f87fda1
openvswitch: 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>
include/uapi/linux/openvswitch.h
net/openvswitch/datapath.c