tomoaki/hakmem
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Fix magazine spill RAW pointer type conversion for Headerless mode

Browse Source 
Problem: bulk_mag_to_sll_if_room() was passing raw pointers directly to
tls_sll_push() without HAK_BASE_FROM_RAW() conversion, causing memory
corruption in Headerless mode where pointer arithmetic expectations differ.

Solution: Add HAK_BASE_FROM_RAW() wrapper before passing to tls_sll_push()

Verification:
- cfrac: PASS (Headerless ON/OFF)
- sh8bench: PASS (Headerless ON/OFF)
- No regressions in existing tests

🤖 Generated with Claude Code

Co-Authored-By: Claude <noreply@anthropic.com>
This commit is contained in:
Moe Charm (CI)
2025-12-03 15:30:28 +09:00
parent 2dc9d5d596
commit f3f75ba3da
1 changed files with 2 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
core/hakmem_tiny_refill.inc.h
View File

@ -225,7 +225,8 @@ static inline int bulk_mag_to_sll_if_room(int class_idx, TinyTLSMag* mag, int n)
int pushed = 0; int pushed = 0;
for (int i = 0; i < take; i++) { for (int i = 0; i < take; i++) {
void* p = mag->items[--mag->top].ptr; void* p = mag->items[--mag->top].ptr;
if (!tls_sll_push(class_idx, p, cap)) { hak_base_ptr_t base_p = HAK_BASE_FROM_RAW(p);
if (!tls_sll_push(class_idx, base_p, cap)) {
mag->top++; // rollback last mag->top++; // rollback last
break; break;
} }