diff --git a/.github/workflows/rust-clippy.yml b/.github/workflows/rust-clippy.yml index c2772fa..3a4b73c 100644 --- a/.github/workflows/rust-clippy.yml +++ b/.github/workflows/rust-clippy.yml @@ -45,8 +45,9 @@ jobs: run: cargo clippy --all-features - --message-format=json | clippy-sarif | tee rust-clippy-results.sarif | sarif-fmt --frozen + --message-format=json | clippy-sarif | tee rust-clippy-results.sarif | sarif-fmt + continue-on-error: true - name: Upload analysis results to GitHub diff --git a/mimalloc/mimalloc-sys/mimalloc/src/arena.c b/mimalloc/mimalloc-sys/mimalloc/src/arena.c index a04a04c..b5b7c09 100644 --- a/mimalloc/mimalloc-sys/mimalloc/src/arena.c +++ b/mimalloc/mimalloc-sys/mimalloc/src/arena.c @@ -623,12 +623,12 @@ void _mi_arena_free(void* p, size_t size, size_t committed_size, mi_memid_t memi // checks if (arena == NULL) { - _mi_error_message(EINVAL, "trying to free from non-existent arena: %p, size %zu, memid: 0x%zx\n", p, size, memid); + _mi_error_message(EINVAL, "trying to free from non-existent arena: %p, size %zu, memid: 0x%zx\n", p, size, (size_t)*(uintptr_t*)&memid); return; } mi_assert_internal(arena->field_count > mi_bitmap_index_field(bitmap_idx)); if (arena->field_count <= mi_bitmap_index_field(bitmap_idx)) { - _mi_error_message(EINVAL, "trying to free from non-existent arena block: %p, size %zu, memid: 0x%zx\n", p, size, memid); + _mi_error_message(EINVAL, "trying to free from non-existent arena block: %p, size %zu, memid: 0x%zx\n", p, size, (size_t)*(uintptr_t*)&memid); return; } diff --git a/mimalloc/mimalloc-sys/mimalloc/src/prim/unix/prim.c b/mimalloc/mimalloc-sys/mimalloc/src/prim/unix/prim.c index 314281f..d37c182 100644 --- a/mimalloc/mimalloc-sys/mimalloc/src/prim/unix/prim.c +++ b/mimalloc/mimalloc-sys/mimalloc/src/prim/unix/prim.c @@ -449,7 +449,7 @@ int _mi_prim_alloc_huge_os_pages(void* hint_addr, size_t size, int numa_node, bo long err = mi_prim_mbind(*addr, size, MPOL_PREFERRED, &numa_mask, 8*MI_INTPTR_SIZE, 0); if (err != 0) { err = errno; - _mi_warning_message("failed to bind huge (1GiB) pages to numa node %d (error: %d (0x%x))\n", numa_node, err, err); + _mi_warning_message("failed to bind huge (1GiB) pages to numa node %d (error: %ld (0x%lx))\n", numa_node, err, err); } } return (*addr != NULL ? 0 : errno); @@ -491,7 +491,7 @@ size_t _mi_prim_numa_node_count(void) { unsigned node = 0; for(node = 0; node < 256; node++) { // enumerate node entries -- todo: it there a more efficient way to do this? (but ensure there is no allocation) - snprintf(buf, 127, "/sys/devices/system/node/node%u", node + 1); + snprintf(buf, sizeof(buf), "/sys/devices/system/node/node%u", node + 1); if (mi_prim_access(buf,R_OK) != 0) break; } return (node+1);