Return value stored in "ret" but it may be overwritten before use.
Add goto to return when meet an error. Issue reported by coverity scan.

Coverity issue: 395532
Fixes: 15f483feec65 ("graph: fix model check in core binding")

Signed-off-by: Zhirun Yan <zhirun....@intel.com>
---
 app/test/test_graph.c | 11 +++++++----
 1 file changed, 7 insertions(+), 4 deletions(-)

diff --git a/app/test/test_graph.c b/app/test/test_graph.c
index af90ac07ec..7995f67757 100644
--- a/app/test/test_graph.c
+++ b/app/test/test_graph.c
@@ -740,13 +740,13 @@ test_graph_model_mcore_dispatch_core_bind_unbind(void)
        ret = rte_graph_worker_model_set(RTE_GRAPH_MODEL_MCORE_DISPATCH);
        if (ret != 0) {
                printf("Set graph mcore dispatch model failed\n");
-               ret = -1;
+               goto fail;
        }
 
        ret = rte_graph_model_mcore_dispatch_core_bind(cloned_graph_id, 
worker_lcore);
        if (ret != 0) {
                printf("bind graph %d to lcore %u failed\n", graph_id, 
worker_lcore);
-               ret = -1;
+               goto fail;
        }
 
        graph = rte_graph_lookup("worker0-cloned-test2");
@@ -755,6 +755,7 @@ test_graph_model_mcore_dispatch_core_bind_unbind(void)
                printf("bind graph %s(id:%d) with lcore %u failed\n",
                       graph->name, graph->id, worker_lcore);
                ret = -1;
+               goto fail;
        }
 
        rte_graph_model_mcore_dispatch_core_unbind(cloned_graph_id);
@@ -764,6 +765,7 @@ test_graph_model_mcore_dispatch_core_bind_unbind(void)
                ret = -1;
        }
 
+fail:
        rte_graph_destroy(cloned_graph_id);
 
        return ret;
@@ -781,7 +783,7 @@ test_graph_worker_model_set_get(void)
        ret = rte_graph_worker_model_set(RTE_GRAPH_MODEL_MCORE_DISPATCH);
        if (ret != 0) {
                printf("Set graph mcore dispatch model failed\n");
-               ret = -1;
+               goto fail;
        }
 
        graph = rte_graph_lookup("worker0-cloned-test3");
@@ -790,9 +792,10 @@ test_graph_worker_model_set_get(void)
                ret = -1;
        }
 
+fail:
        rte_graph_destroy(cloned_graph_id);
 
-       return 0;
+       return ret;
 }
 
 static int
-- 
2.37.2

Reply via email to