diff options
author | Ben Pfaff <blp@nicira.com> | 2011-10-24 15:18:49 -0700 |
---|---|---|
committer | Ben Pfaff <blp@nicira.com> | 2011-10-24 15:45:59 -0700 |
commit | 6d0840ac87f147da1e28fcf9a9ffbf9a081fa599 (patch) | |
tree | a54f1afcfe20dc9b0d24253ead35bbe9f22d5e39 /datapath | |
parent | a5f1b8f89e6bc8210e8eba35430ee9e31f7a674f (diff) |
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 <blp@nicira.com>
Acked-by: Jesse Gross <jesse@nicira.com>
Diffstat (limited to 'datapath')
-rw-r--r-- | datapath/datapath.c | 2 |
1 files changed, 1 insertions, 1 deletions
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; |