diff options
author | Bill Fischofer <bill.fischofer@linaro.org> | 2016-05-26 18:46:43 -0500 |
---|---|---|
committer | Maxim Uvarov <maxim.uvarov@linaro.org> | 2016-05-27 18:32:54 +0300 |
commit | 5171b8109852ec2780e51a480a8e9a08e5cc892b (patch) | |
tree | 62734722c4c2d10a4dd854858ec78753f854ec11 /test | |
parent | 77fc7fc9d26e57023b73131ad2269fb4915d21e5 (diff) |
validation: tm: handle pktio open/config failures properly
Add proper sequencing of checks for pktio open / configure failures.
Signed-off-by: Bill Fischofer <bill.fischofer@linaro.org>
Signed-off-by: Maxim Uvarov <maxim.uvarov@linaro.org>
Diffstat (limited to 'test')
-rw-r--r-- | test/validation/traffic_mngr/traffic_mngr.c | 12 |
1 files changed, 6 insertions, 6 deletions
diff --git a/test/validation/traffic_mngr/traffic_mngr.c b/test/validation/traffic_mngr/traffic_mngr.c index 2bbcc1dd8..e099083be 100644 --- a/test/validation/traffic_mngr/traffic_mngr.c +++ b/test/validation/traffic_mngr/traffic_mngr.c @@ -492,19 +492,19 @@ static int open_pktios(void) &pktio_param); if (pktio == ODP_PKTIO_INVALID) pktio = odp_pktio_lookup(iface_name[iface]); + if (pktio == ODP_PKTIO_INVALID) { + LOG_ERR("odp_pktio_open() failed\n"); + return -1; + } /* Set defaults for PktIn and PktOut queues */ - odp_pktin_queue_config(pktio, NULL); - odp_pktout_queue_config(pktio, NULL); + (void)odp_pktin_queue_config(pktio, NULL); + (void)odp_pktout_queue_config(pktio, NULL); rc = odp_pktio_promisc_mode_set(pktio, true); if (rc != 0) printf("****** promisc_mode_set failed ******\n"); pktios[iface] = pktio; - if (pktio == ODP_PKTIO_INVALID) { - LOG_ERR("odp_pktio_open() failed\n"); - return -1; - } if (odp_pktin_queue(pktio, &pktins[iface], 1) != 1) { odp_pktio_close(pktio); |