On 03/24/2014 07:56 PM, Jarno Rajahalme wrote:
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 <[email protected]>
v4: Maintain OVS 1.7 semantics: Allow missing actions on OVS_FLOW_CMD_SET,
which cause the existing actions to be retained.
LGTM
Acked-by: Thomas Graf <[email protected]>
_______________________________________________
dev mailing list
[email protected]
http://openvswitch.org/mailman/listinfo/dev
