[ovs-dev] [PATCH] meta-flow: Fix wrong parentheses.
Kmindg
kmindg at gmail.com
Thu Feb 12 13:47:32 UTC 2015
Bitwise not operator has higher precedence than Bitwise right shift
operator, so the MFF_IP_DSCP_SHIFTED case always returns true currently.
Signed-off-by: Kmindg <kmindg at gmail.com>
---
lib/meta-flow.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/lib/meta-flow.c b/lib/meta-flow.c
index 9ce4cfe..6576336 100644
--- a/lib/meta-flow.c
+++ b/lib/meta-flow.c
@@ -416,7 +416,7 @@ mf_is_value_valid(const struct mf_field *mf, const union mf_value *value)
case MFF_IP_DSCP:
return !(value->u8 & ~IP_DSCP_MASK);
case MFF_IP_DSCP_SHIFTED:
- return !(value->u8 & (~IP_DSCP_MASK >> 2));
+ return !(value->u8 & ~(IP_DSCP_MASK >> 2));
case MFF_IP_ECN:
return !(value->u8 & ~IP_ECN_MASK);
case MFF_IP_FRAG:
--
2.3.0
More information about the dev
mailing list