[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