Skip to content

Commit a714e45

Browse files
committed
Fix memory leak in the testNumaSplit:checkModeSplit test
Signed-off-by: Lukasz Dorau <[email protected]>
1 parent e240489 commit a714e45

File tree

1 file changed

+5
-5
lines changed

1 file changed

+5
-5
lines changed

test/provider_os_memory_multiple_numa_nodes.cpp

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -674,17 +674,17 @@ TEST_P(testNumaSplit, checkModeSplit) {
674674
auto [required_numa_nodes, pages, in, out] = param;
675675
umf_result_t umf_result;
676676

677-
umf_os_memory_provider_params_handle_t os_memory_provider_params = nullptr;
678-
679-
umf_result = umfOsMemoryProviderParamsCreate(&os_memory_provider_params);
680-
ASSERT_EQ(umf_result, UMF_RESULT_SUCCESS);
681-
682677
std::vector<unsigned> numa_nodes = get_available_numa_nodes();
683678

684679
if (numa_nodes.size() < required_numa_nodes) {
685680
GTEST_SKIP_("Not enough numa nodes");
686681
}
687682

683+
umf_os_memory_provider_params_handle_t os_memory_provider_params = nullptr;
684+
685+
umf_result = umfOsMemoryProviderParamsCreate(&os_memory_provider_params);
686+
ASSERT_EQ(umf_result, UMF_RESULT_SUCCESS);
687+
688688
ASSERT_EQ(out.size(), pages)
689689
<< "Wrong test input - out array size doesn't match page count";
690690

0 commit comments

Comments
 (0)