The if/else branches both return, making the trailing 'return 0'
unreachable dead code.  Remove it.

Found by OpenScanHub Coverity (DEADCODE).
Signed-off-by: Timothy Redaelli <[email protected]>
---
 lib/ofp-table.c | 2 --
 1 file changed, 2 deletions(-)

diff --git a/lib/ofp-table.c b/lib/ofp-table.c
index 0435df45b..59b417a00 100644
--- a/lib/ofp-table.c
+++ b/lib/ofp-table.c
@@ -2392,8 +2392,6 @@ ofputil_decode_table_status(const struct ofp_header *oh,
     } else {
         return OFPERR_OFPBRC_BAD_VERSION;
     }
-
-    return 0;
 }
 
 void
-- 
2.54.0

_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to