mirrors/ladybird
0
0
Fork 0
mirror of https://github.com/LadybirdBrowser/ladybird.git synced 2025-08-08 01:00:05 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 9

Kernel: Fix assertion statement in GenericInterruptHandler

Browse source 
We need to assert if interrupts are not disabled when changing the
interrupt number of an interrupt handler.
Before this fix, any change like this would lead to a crash,
because we are using InterruptDisabler in IRQHandler::change_irq_number.
This commit is contained in:
Liav A 2020-09-19 14:11:15 +03:00 • committed by Andreas Kling
commit 82b0171812
Notes: sideshowbarker 2024-07-19 02:20:09 +09:00
1 changed files with 1 additions and 1 deletions
Download patch file Download diff file Expand all files Collapse all files

2
Kernel/Interrupts/GenericInterruptHandler.cpp
View file

@ -56,7 +56,7 @@ GenericInterruptHandler::~GenericInterruptHandler()
void GenericInterruptHandler::change_interrupt_number(u8 number) void GenericInterruptHandler::change_interrupt_number(u8 number)
{ {
ASSERT_INTERRUPTS_ENABLED(); ASSERT_INTERRUPTS_DISABLED();
ASSERT(!m_disable_remap); ASSERT(!m_disable_remap);
unregister_generic_interrupt_handler(InterruptManagement::acquire_mapped_interrupt_number(interrupt_number()), *this); unregister_generic_interrupt_handler(InterruptManagement::acquire_mapped_interrupt_number(interrupt_number()), *this);
m_interrupt_number = number; m_interrupt_number = number;