d/numam-dpdk
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

test/timer: check memzone allocation

Browse Source 
Segmentation fault may occur without checking if memzone
reserves succeed or not.

Fixes: 50247fe03fe0 ("test/timer: exercise new APIs in secondary process")
Cc: stable@dpdk.org

Signed-off-by: Min Hu (Connor) <humin29@huawei.com>
Acked-by: Erik Gabriel Carrillo <erik.g.carrillo@intel.com>
This commit is contained in:
Min Hu (Connor) 2021-05-04 09:07:49 +08:00 committed by Thomas Monjalon
parent 80aa15c4df
commit fb9b862e7c
1 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
app/test/test_timer_secondary.c
View File

@ -125,9 +125,9 @@ test_timer_secondary(void)
mz = rte_memzone_reserve(TEST_INFO_MZ_NAME, sizeof(*test_info),
SOCKET_ID_ANY, 0);
test_info = mz->addr;
TEST_ASSERT_NOT_NULL(test_info, "Couldn't allocate memory for "
TEST_ASSERT_NOT_NULL(mz, "Couldn't allocate memory for "
"test data");
test_info = mz->addr;
test_info->tim_mempool = rte_mempool_create("test_timer_mp",
NUM_TIMERS, sizeof(struct rte_timer), 0, 0,
@ -171,9 +171,9 @@ test_timer_secondary(void)
int i;
mz = rte_memzone_lookup(TEST_INFO_MZ_NAME);
test_info = mz->addr;
TEST_ASSERT_NOT_NULL(test_info, "Couldn't lookup memzone for "
TEST_ASSERT_NOT_NULL(mz, "Couldn't lookup memzone for "
"test info");
test_info = mz->addr;
for (i = 0; i < NUM_TIMERS; i++) {
rte_mempool_get(test_info->tim_mempool, (void **)&tim);