From 0135dc8b4e91886b0e9f5c3e1c3e973d9ee575c7 Mon Sep 17 00:00:00 2001 From: Ben Pfaff Date: Tue, 20 Nov 2012 07:46:41 -0800 Subject: [PATCH] dpif-netdev: Make odp_flow_key_to_flow() check more explicit. ODP_FIT_PERFECT has value 0 but this 'if' statement doesn't make much sense unless you happen to know that. Signed-off-by: Ben Pfaff Acked-by: Kyle Mestery --- lib/dpif-netdev.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/lib/dpif-netdev.c b/lib/dpif-netdev.c index cd4ab282..92108df1 100644 --- a/lib/dpif-netdev.c +++ b/lib/dpif-netdev.c @@ -670,7 +670,7 @@ static int dpif_netdev_flow_from_nlattrs(const struct nlattr *key, uint32_t key_len, struct flow *flow) { - if (odp_flow_key_to_flow(key, key_len, flow)) { + if (odp_flow_key_to_flow(key, key_len, flow) != ODP_FIT_PERFECT) { /* This should not happen: it indicates that odp_flow_key_from_flow() * and odp_flow_key_to_flow() disagree on the acceptable form of a * flow. Log the problem as an error, with enough details to enable -- 2.30.2