Bananymous
/
banan-os
1
1
Fork 1
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Kernel: Fix signal delivery without an alternate stack 

I had only tested that sigaltstack worked, so I didn't notice my normal
signals broke :D
This commit is contained in:
Bananymous 2025-08-25 22:15:22 +03:00
parent f926e599fa
commit 10bd24e585
1 changed files with 17 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

23
kernel/kernel/Thread.cpp
View File

@ -573,13 +573,24 @@ namespace Kernel
{ {
// Make sure stack is allocated // Make sure stack is allocated
const vaddr_t pages[3] { vaddr_t pages[3] {};
(interrupt_stack.sp - sizeof(uintptr_t)) & PAGE_ADDR_MASK, size_t page_count { 0 };
(signal_stack_top - 4 * sizeof(uintptr_t)) & PAGE_ADDR_MASK,
(signal_stack_top - 1 * sizeof(uintptr_t)) & PAGE_ADDR_MASK,
};
for (size_t i = 0; i < 3; i++) if (signal_stack_top == 0)
{
pages[0] = (interrupt_stack.sp - 1 * sizeof(uintptr_t)) & PAGE_ADDR_MASK;
pages[1] = (interrupt_stack.sp - 5 * sizeof(uintptr_t)) & PAGE_ADDR_MASK;
page_count = 2;
}
else
{
pages[0] = (interrupt_stack.sp - sizeof(uintptr_t)) & PAGE_ADDR_MASK;
pages[1] = (signal_stack_top - 4 * sizeof(uintptr_t)) & PAGE_ADDR_MASK;
pages[2] = (signal_stack_top - 1 * sizeof(uintptr_t)) & PAGE_ADDR_MASK;
page_count = 3;
}
for (size_t i = 0; i < page_count; i++)
{ {
if (m_process->page_table().get_page_flags(pages[i]) & PageTable::Flags::Present) if (m_process->page_table().get_page_flags(pages[i]) & PageTable::Flags::Present)
continue; continue;