Merged both patches.
Maxim.
On 04/27/2015 19:48, Stuart Haslam wrote:
Coverity complains that the return value of odp_pktio_close() isn't
being checked.
This fixes https://bugs.linaro.org/show_bug.cgi?id=1513
Signed-off-by: Stuart Haslam <[email protected]>
---
test/performance/odp_pktio_perf.c | 15 ++++++++++++---
1 file changed, 12 insertions(+), 3 deletions(-)
diff --git a/test/performance/odp_pktio_perf.c
b/test/performance/odp_pktio_perf.c
index 0be8d46..e24ceb5 100644
--- a/test/performance/odp_pktio_perf.c
+++ b/test/performance/odp_pktio_perf.c
@@ -746,10 +746,19 @@ static int test_term(void)
int i;
int ret = 0;
- if (gbl_args->pktio_tx != gbl_args->pktio_rx)
- odp_pktio_close(gbl_args->pktio_tx);
+ if (gbl_args->pktio_tx != gbl_args->pktio_rx) {
+ if (odp_pktio_close(gbl_args->pktio_tx) != 0) {
+ LOG_ERR("Failed to close pktio_tx\n");
+ ret = -1;
+ }
+ }
+
destroy_inq(gbl_args->pktio_rx);
- odp_pktio_close(gbl_args->pktio_rx);
+
+ if (odp_pktio_close(gbl_args->pktio_rx) != 0) {
+ LOG_ERR("Failed to close pktio_rx\n");
+ ret = -1;
+ }
for (i = 0; i < gbl_args->args.num_ifaces; ++i) {
snprintf(pool_name, sizeof(pool_name),
_______________________________________________
lng-odp mailing list
[email protected]
https://lists.linaro.org/mailman/listinfo/lng-odp
