Skip to content

Commit cb0dba0

Browse files
authored
Merge pull request #972 from ldorau/Ignore_return_value_of__in_an_error_handling_path
Ignore return value of `je_mallctl()` in an error handling path
2 parents 77f0cb5 + 7d1f1d0 commit cb0dba0

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

src/pool/pool_jemalloc.c

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -504,7 +504,7 @@ static umf_result_t op_initialize(umf_memory_provider_handle_t provider,
504504
err = je_mallctl(cmd, NULL, NULL, (void *)&pHooks, sizeof(void *));
505505
if (err) {
506506
snprintf(cmd, sizeof(cmd), "arena.%u.destroy", arena_index);
507-
je_mallctl(cmd, NULL, 0, NULL, 0);
507+
(void)je_mallctl(cmd, NULL, 0, NULL, 0);
508508
LOG_ERR("Could not setup extent_hooks for newly created arena.");
509509
goto err_free_pool;
510510
}
@@ -528,7 +528,7 @@ static void op_finalize(void *pool) {
528528
jemalloc_memory_pool_t *je_pool = (jemalloc_memory_pool_t *)pool;
529529
char cmd[64];
530530
snprintf(cmd, sizeof(cmd), "arena.%u.destroy", je_pool->arena_index);
531-
je_mallctl(cmd, NULL, 0, NULL, 0);
531+
(void)je_mallctl(cmd, NULL, 0, NULL, 0);
532532
pool_by_arena_index[je_pool->arena_index] = NULL;
533533
umf_ba_global_free(je_pool);
534534

0 commit comments

Comments
 (0)