examples/bbdev: fix unchecked return of stats
authorAmr Mokhtar <amr.mokhtar@intel.com>
Wed, 31 Jan 2018 14:40:25 +0000 (14:40 +0000)
committerThomas Monjalon <thomas@monjalon.net>
Wed, 31 Jan 2018 17:51:02 +0000 (18:51 +0100)
Added a check on rte_bbdev_stats_get() return before
printing out the statistics results.

Coverity issue: 257018
Fixes: 1ffee690eaa1 ("examples/bbdev: add sample app")

Signed-off-by: Amr Mokhtar <amr.mokhtar@intel.com>
examples/bbdev_app/main.c

index e779db2..3c45256 100644 (file)
@@ -616,8 +616,16 @@ print_stats(struct stats_lcore_params *stats_lcore)
                                        xstats[i].value);
        }
 
+       ret = rte_bbdev_stats_get(bbdev_id, &bbstats);
+       if (ret < 0) {
+               free(xstats);
+               free(xstats_names);
+               rte_exit(EXIT_FAILURE,
+                               "ERROR(%d): Failure to get BBDEV %u statistics\n",
+                               ret, bbdev_id);
+       }
+
        printf("\nBBDEV STATISTICS:\n=================\n");
-       rte_bbdev_stats_get(bbdev_id, &bbstats);
        printf("BBDEV %u: %s enqueue count:\t\t%"PRIu64"\n",
                        bbdev_id, stats_border,
                        bbstats.enqueued_count);