Skip to content

Allocator reference#693

Merged
arvidn merged 2 commits intomainfrom
allocator-reference
Feb 8, 2026
Merged

Allocator reference#693
arvidn merged 2 commits intomainfrom
allocator-reference

Conversation

@arvidn
Copy link
Copy Markdown
Contributor

@arvidn arvidn commented Feb 7, 2026

Use 32 bits for version in allocator-debug feature. This allows 2^32 uses of restore_checkpoint() on the Allocator when debugging.


Note

Low Risk
Debug-only behavior changes plus fuzz configuration; no changes to non-debug allocator semantics, but could affect assertions/panics when allocator-debug is enabled.

Overview
Improves allocator-debug safety checks by separating AllocatorReference into a 24-bit-ish fingerprint plus a full 32-bit version, removing the previous 8-bit version packing limit and allowing far more restore_checkpoint() cycles before debug invalidation tracking breaks.

Updates NodePtr creation and validate_node() to use the new explicit version field, and enables the allocator-debug feature in the fuzz crate to exercise these debug checks during fuzzing.

Written by Cursor Bugbot for commit a741578. This will update automatically on new commits. Configure here.

…-debug feature more robust (but more memory hungry)
@arvidn arvidn requested a review from richardkiss February 7, 2026 09:03
@coveralls-official
Copy link
Copy Markdown

coveralls-official bot commented Feb 7, 2026

Pull Request Test Coverage Report for Build 21778821045

Details

  • 3 of 4 (75.0%) changed or added relevant lines in 1 file are covered.
  • 12 unchanged lines in 1 file lost coverage.
  • Overall coverage increased (+0.01%) to 90.474%

Changes Missing Coverage Covered Lines Changed/Added Lines %
src/allocator.rs 3 4 75.0%
Files with Coverage Reduction New Missed Lines %
src/allocator.rs 12 96.22%
Totals Coverage Status
Change from base Build 21748841115: 0.01%
Covered Lines: 6582
Relevant Lines: 7275

💛 - Coveralls

@arvidn arvidn force-pushed the allocator-reference branch from 599df40 to a741578 Compare February 7, 2026 10:43
@arvidn arvidn merged commit a1a721f into main Feb 8, 2026
32 checks passed
@arvidn arvidn deleted the allocator-reference branch February 8, 2026 09:26
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.

2 participants