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. --- build/src/kernel/initcode.o | Bin 31496 -> 31528 bytes build/src/kernel/mm/vm.o | Bin 234008 -> 234008 bytes build/src/kernel/proc/exec.o | Bin 62896 -> 64304 bytes build/src/kernel/proc/proc.o | Bin 221432 -> 225240 bytes build/src/kernel/trap/handler.o | Bin 71232 -> 72624 bytes 5 files changed, 0 insertions(+), 0 deletions(-) (limited to 'build/src/kernel') diff --git a/build/src/kernel/initcode.o b/build/src/kernel/initcode.o index c575853..cb2dd53 100644 Binary files a/build/src/kernel/initcode.o and b/build/src/kernel/initcode.o differ diff --git a/build/src/kernel/mm/vm.o b/build/src/kernel/mm/vm.o index d73e9a6..96840aa 100644 Binary files a/build/src/kernel/mm/vm.o and b/build/src/kernel/mm/vm.o differ diff --git a/build/src/kernel/proc/exec.o b/build/src/kernel/proc/exec.o index 5b735ed..26a77b6 100644 Binary files a/build/src/kernel/proc/exec.o and b/build/src/kernel/proc/exec.o differ diff --git a/build/src/kernel/proc/proc.o b/build/src/kernel/proc/proc.o index 7ae741b..940c575 100644 Binary files a/build/src/kernel/proc/proc.o and b/build/src/kernel/proc/proc.o differ diff --git a/build/src/kernel/trap/handler.o b/build/src/kernel/trap/handler.o index 44a2baa..bd42409 100644 Binary files a/build/src/kernel/trap/handler.o and b/build/src/kernel/trap/handler.o differ -- cgit v1.2.3