From 10614e04929a6d109a5e90524e446b637aaf0c13 Mon Sep 17 00:00:00 2001 From: Richard Qin Date: Sat, 3 Jan 2026 02:13:08 +0800 Subject: Fix: critical malloc size calculation bug Summary: - Fixed an integer arithmetic error in malloc() where 'nunits' was initialized to 0, causing incorrect heap sizing. - This bug caused malloc to return pointers pointing to unmapped memory pages, leading to Store Page Fault (scause 15) at process boundaries. - Switched memory allocator logic to use byte-based size calculation instead of K&R style units to prevent off-by-one errors. Improvements: - Shell now starts successfully without segfaults. --- include/kernel/proc_def.h | 4 ++++ 1 file changed, 4 insertions(+) (limited to 'include/kernel/proc_def.h') diff --git a/include/kernel/proc_def.h b/include/kernel/proc_def.h index 06c80aa..5ce4c03 100644 --- a/include/kernel/proc_def.h +++ b/include/kernel/proc_def.h @@ -59,6 +59,10 @@ struct Proc struct Proc *next; struct Proc *prev; + + struct Proc *all_next; + struct Proc *all_prev; + int priority; int ticks; struct cpu *cpu; -- cgit v1.2.3