Skip to content

Commit

Permalink
test/bonding: add missing check
Browse files Browse the repository at this point in the history
[ upstream commit 0ecafc2 ]

CI found the function without checking return value in this place.

Coverity issue: 403101
Fixes: 92073ef ("bond: unit tests")

Signed-off-by: Long Wu <long.wu@corigine.com>
Reviewed-by: Chaoyong He <chaoyong.he@corigine.com>
Reviewed-by: Peng Zhang <peng.zhang@corigine.com>
Acked-by: Ferruh Yigit <ferruh.yigit@amd.com>
  • Loading branch information
wulong2022 authored and bluca committed Nov 8, 2023
1 parent 9b198fc commit 894b686
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion app/test/test_link_bonding.c
Original file line number Diff line number Diff line change
Expand Up @@ -445,7 +445,8 @@ test_add_already_bonded_slave_to_bonded_device(void)
uint16_t slaves[RTE_MAX_ETHPORTS];
char pmd_name[RTE_ETH_NAME_MAX_LEN];

test_add_slave_to_bonded_device();
TEST_ASSERT_SUCCESS(test_add_slave_to_bonded_device(),
"Failed to add member to bonding device");

current_slave_count = rte_eth_bond_slaves_get(test_params->bonded_port_id,
slaves, RTE_MAX_ETHPORTS);
Expand Down

0 comments on commit 894b686

Please sign in to comment.