From 6d0840ac87f147da1e28fcf9a9ffbf9a081fa599 Mon Sep 17 00:00:00 2001 From: Ben Pfaff Date: Mon, 24 Oct 2011 15:18:49 -0700 Subject: [PATCH] datapath: Fully parenthesize ACTION macro. This doesn't matter for any of the current users of ACTION, nor do I expect it to matter, but it seems like a good idea nonetheless. Signed-off-by: Ben Pfaff Acked-by: Jesse Gross --- datapath/datapath.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/datapath/datapath.c b/datapath/datapath.c index dab86313..be90d549 100644 --- a/datapath/datapath.c +++ b/datapath/datapath.c @@ -552,7 +552,7 @@ static int validate_action_key(const struct nlattr *a, nla_len(ovs_key) != ovs_key_lens[key_type]) return -EINVAL; -#define ACTION(act, key) ((act << 8) | key) +#define ACTION(act, key) (((act) << 8) | (key)) switch(ACTION(act_type, key_type)) { const struct ovs_key_ipv4 *ipv4_key; -- 2.30.2