Coverity CID 279497 reports "Operands don't affect result". Because flow->ct_state is uint8_t and DP_NETDEV_CS_UNSUPPORTED_MASK is '0xffffff00'. So remove the statement.
Cc: Usman Ansari <[email protected]> Signed-off-by: William Tu <[email protected]> --- lib/dpif-netdev.c | 4 ---- 1 file changed, 4 deletions(-) diff --git a/lib/dpif-netdev.c b/lib/dpif-netdev.c index a798db45d9cb..0e2678d002d5 100644 --- a/lib/dpif-netdev.c +++ b/lib/dpif-netdev.c @@ -3224,10 +3224,6 @@ dpif_netdev_flow_from_nlattrs(const struct nlattr *key, uint32_t key_len, return EINVAL; } - if (flow->ct_state & DP_NETDEV_CS_UNSUPPORTED_MASK) { - return EINVAL; - } - return 0; } -- 2.7.4 _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
