Kernel: Fix PID/TID confusion in send_signal

This fixes the issue of a specific type of unkillable processes.

1 files changed, 14 insertions, 3 deletions

diff --git a/Kernel/Process.cpp b/Kernel/Process.cpp
index d286949cb0..cb5bd298c5 100644
--- a/Kernel/Process.cpp
+++ b/Kernel/Process.cpp
@@ -790,9 +790,20 @@ void Process::terminate_due_to_signal(u8 signal)
 KResult Process::send_signal(u8 signal, Process* sender)
 {
     InterruptDisabler disabler;
-    // FIXME: PID/TID BUG
-    if (auto* thread = Thread::from_tid(m_pid.value())) {
-        thread->send_signal(signal, sender);
+    Thread* receiver_thread;
+    // Try to send it to the "obvious" main thread:
+    receiver_thread = Thread::from_tid(m_pid.value());
+    // If the main thread has died, there may still be other threads:
+    if (!receiver_thread) {
+        // The first one should be good enough.
+        // Neither kill(2) nor kill(3) specify any selection precedure.
+        for_each_thread([&receiver_thread](Thread& thread) -> IterationDecision {
+            receiver_thread = &thread;
+            return IterationDecision::Break;
+        });
+    }
+    if (receiver_thread) {
+        receiver_thread->send_signal(signal, sender);
         return KSuccess;
     }
     return KResult(-ESRCH);