Kernel: Make copy_{from,to}_user() return KResult and use TRY()

This makes EFAULT propagation flow much more naturally. :^)

1 files changed, 3 insertions, 2 deletions

diff --git a/Kernel/Thread.cpp b/Kernel/Thread.cpp
index e75d8a387b..78852033a1 100644
--- a/Kernel/Thread.cpp
+++ b/Kernel/Thread.cpp
@@ -802,10 +802,11 @@ bool Thread::has_signal_handler(u8 signal) const
     return !action.handler_or_sigaction.is_null();
 }
 
-static bool push_value_on_user_stack(FlatPtr& stack, FlatPtr data)
+static void push_value_on_user_stack(FlatPtr& stack, FlatPtr data)
 {
     stack -= sizeof(FlatPtr);
-    return copy_to_user((FlatPtr*)stack, &data);
+    auto result = copy_to_user((FlatPtr*)stack, &data);
+    VERIFY(result.is_success());
 }
 
 void Thread::resume_from_stopped()