Skip to content

Commit 47db5b0

Browse files
authored
Merge pull request #1495 from ldorau/Add_asserts_to_verify_sysconf_result
Add asserts to verify sysconf() result
2 parents 099cc85 + fd033c4 commit 47db5b0

File tree

1 file changed

+2
-0
lines changed

1 file changed

+2
-0
lines changed

test/provider_os_memory_multiple_numa_nodes.cpp

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -255,6 +255,7 @@ TEST_P(testNumaOnEachNode, checkModeInterleaveSingleNode) {
255255

256256
constexpr int pages_num = 1024;
257257
size_t page_size = sysconf(_SC_PAGE_SIZE);
258+
ASSERT_GT(page_size, 0);
258259
umf_result_t umf_result;
259260
umf_os_memory_provider_params_handle_t os_memory_provider_params = nullptr;
260261

@@ -427,6 +428,7 @@ TEST_F(testNuma, checkModeDefault) {
427428
TEST_F(testNuma, checkModeInterleave) {
428429
constexpr int pages_num = 1024;
429430
size_t page_size = sysconf(_SC_PAGE_SIZE);
431+
ASSERT_GT(page_size, 0);
430432
umf_result_t umf_result;
431433
umf_os_memory_provider_params_handle_t os_memory_provider_params = nullptr;
432434

0 commit comments

Comments
 (0)