xenia/fox32os
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

kernel/allocator: FINALLY fix a long-standing bug with memory freeing

Browse Source 
This fixes issue #7
This commit is contained in:
Ry 2023-07-11 23:37:11 -07:00
parent e647c5835b
commit 460b0be997
1 changed files with 4 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
kernel/allocator.asm
View File

@ -41,6 +41,8 @@ allocate_memory:
push r11 push r11
mov [block], [free_list_head] mov [block], [free_list_head]
cmp [block], 0
ifz jmp allocate_memory_bad
; r10: real_size = requested size + header size ; r10: real_size = requested size + header size
mov r10, r0 mov r10, r0
@ -58,6 +60,7 @@ allocate_memory_while_block:
cmp [block], 0 cmp [block], 0
ifnz jmp allocate_memory_while_block ifnz jmp allocate_memory_while_block
allocate_memory_bad:
; if we reach this point, no good blocks were found ; if we reach this point, no good blocks were found
mov r0, 0 mov r0, 0
@ -177,7 +180,7 @@ free_memory:
call block_set_next call block_set_next
cmp [free_list_head], 0 cmp [free_list_head], 0
ifnz mov r1, r0 ifnz mov r1, r2
ifnz mov r0, [free_list_head] ifnz mov r0, [free_list_head]
ifnz call block_set_prev ifnz call block_set_prev