Commit 5829e62a authored Sep 11, 2017 by Christophe JAILLET Committed by David S. Miller Sep 12, 2017

openvswitch: Fix an error handling path in 'ovs_nla_init_match_and_action()'



All other error handling paths in this function go through the 'error'
label. This one should do the same.

Fixes: 9cc9a5cb ("datapath: Avoid using stack larger than 1024.")
Signed-off-by: Christophe JAILLET <christophe.jaillet@wanadoo.fr>
Acked-by: Pravin B Shelar <pshelar@ovn.org>
Signed-off-by: David S. Miller <davem@davemloft.net>

parent 9c082731

net/openvswitch/datapath.c

+2 −1

Original line number	Diff line number	Diff line
		@@ -1112,7 +1112,8 @@ static int ovs_nla_init_match_and_action(struct net *net,
		if (!a[OVS_FLOW_ATTR_KEY]) {
		OVS_NLERR(log,
		"Flow key attribute not present in set flow.");
		return -EINVAL;
		error = -EINVAL;
		goto error;
		}

		*acts = get_flow_actions(net, a[OVS_FLOW_ATTR_ACTIONS], key,