beenull/ladybird
1
0
Fork 0
mirror of https://github.com/LadybirdBrowser/ladybird.git synced 2024-11-22 15:40:19 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 6

Kernel: Validate the sys$alarm signal send always succeeds

Browse source 
Previously we were ignoring this return code, instead use MUST(..)
to make sure it always succeeds.
This commit is contained in:
Brian Gianforcaro 2022-08-09 20:31:29 -07:00
parent 00936e151e
commit 09d5360be3
Notes: sideshowbarker 2024-07-17 08:20:53 +09:00 
Author: https://github.com/bgianfo
Commit: https://github.com/SerenityOS/serenity/commit/09d5360be3
Pull-request: https://github.com/SerenityOS/serenity/pull/14818
Reviewed-by: https://github.com/kleinesfilmroellchen ✅
1 changed files with 1 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
Kernel/Syscalls/alarm.cpp
View file

@ -34,7 +34,7 @@ ErrorOr<FlatPtr> Process::sys$alarm(unsigned seconds)
m_alarm_timer = TRY(adopt_nonnull_ref_or_enomem(new (nothrow) Timer));
}
auto timer_was_added = TimerQueue::the().add_timer_without_id(*m_alarm_timer, CLOCK_REALTIME_COARSE, deadline, [this]() {
[[maybe_unused]] auto rc = send_signal(SIGALRM, nullptr);
MUST(send_signal(SIGALRM, nullptr));
});
if (!timer_was_added)
return ENOMEM;