Skip to content

chore(deps): update RocksDB binaries and bindings to 11.8.1 - #12

Closed
github-actions[bot] wants to merge 1 commit into
feature/unified-allocatorfrom
feature/rocksdb-build-22-1
Closed

chore(deps): update RocksDB binaries and bindings to 11.8.1#12
github-actions[bot] wants to merge 1 commit into
feature/unified-allocatorfrom
feature/rocksdb-build-22-1

Conversation

@github-actions

@github-actions github-actions Bot commented Sep 4, 2026

Copy link
Copy Markdown
Contributor

Built and generated from RocksDB v11.8.1.

@kamilchodola

Copy link
Copy Markdown

Closing: built and published to staging as Nethermind.RocksDbBindings 11.8.1-preview.112 (jemalloc=complete); the package is immutable on the feed so this branch is no longer needed.

This is the first build where one allocator owns every allocation inside librocksdb.so. The unprefixed attempt in #9 segfaulted because rocksdb frees glibc-allocated memory in two places: strdup in the C API's SaveError, and getline in PosixHelper::GetQueueSysfsFileValueOfFd (via RegisterDbPaths, on every DB::Open). Symbol analysis shows those plus getcwd(NULL)/realpath(NULL) are the complete set of caller-owned-heap libc imports, so all four are --wrapped onto the bundled allocator and Verify asserts none remains imported. 345/345 on all six platforms.

One caution recorded for reviewers: the getline path only runs where sysfs exposes /sys/dev/block/*/queue/logical_block_size, so a passing suite on a host without it (e.g. the arm64 GitHub runner) is not evidence of safety - the symbol assertion is.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant