ed/Odin
1
0
Fork 0
mirror of https://github.com/Ed94/Odin.git synced 2026-08-06 23:58:50 +00:00
Code Issues Packages Projects Releases Wiki Activity

mem: Panic when passing invalid pointers to small stack free/resize

Browse Source 
This is consistent with `Stack_Allocator`.
This commit is contained in:
Feoramund
2025-06-15 12:34:15 -04:00
parent d7e98ba82a
commit 47a54dd11a
1 changed files with 2 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files
core/mem/allocators.odin
+2 -4
View File
@@ -1387,8 +1387,7 @@ small_stack_free :: proc(
end := start + uintptr(len(s.data)) end := start + uintptr(len(s.data))
curr_addr := uintptr(old_memory) curr_addr := uintptr(old_memory)
if !(start <= curr_addr && curr_addr < end) { if !(start <= curr_addr && curr_addr < end) {
// panic("Out of bounds memory address passed to stack allocator (free)"); panic("Out of bounds memory address passed to small stack allocator (free)", loc)
return .Invalid_Pointer
} }
if curr_addr >= start+uintptr(s.offset) { if curr_addr >= start+uintptr(s.offset) {
// NOTE(bill): Allow double frees // NOTE(bill): Allow double frees
@@ -1543,8 +1542,7 @@ small_stack_resize_bytes_non_zeroed :: proc(
end := start + uintptr(len(s.data)) end := start + uintptr(len(s.data))
curr_addr := uintptr(old_memory) curr_addr := uintptr(old_memory)
if !(start <= curr_addr && curr_addr < end) { if !(start <= curr_addr && curr_addr < end) {
// panic("Out of bounds memory address passed to stack allocator (resize)"); panic("Out of bounds memory address passed to small stack allocator (resize)", loc)
return nil, .Invalid_Pointer
} }
if curr_addr >= start+uintptr(s.offset) { if curr_addr >= start+uintptr(s.offset) {
// NOTE(bill): Treat as a double free // NOTE(bill): Treat as a double free