The two if check does the same task, so removing one. Coverity Issue: 355669 Fixes: 9ad50c29d01d ("examples/ipsec-secgw: add app mode worker")
Signed-off-by: Ankur Dwivedi <adwiv...@marvell.com> --- examples/ipsec-secgw/ipsec_worker.c | 7 +------ 1 file changed, 1 insertion(+), 6 deletions(-) diff --git a/examples/ipsec-secgw/ipsec_worker.c b/examples/ipsec-secgw/ipsec_worker.c index 5fde667..b6c851f 100644 --- a/examples/ipsec-secgw/ipsec_worker.c +++ b/examples/ipsec-secgw/ipsec_worker.c @@ -112,12 +112,7 @@ rte_acl_classify((struct rte_acl_ctx *)sp, &nlp, &res, 1, DEFAULT_MAX_CATEGORIES); - if (unlikely(res == 0)) { - /* No match */ - return 0; - } - - if (res == DISCARD) + if (unlikely(res == DISCARD)) return 0; else if (res == BYPASS) { *sa_idx = -1; -- 1.9.3