From 2d2ea05c97e97848f431b5c9e5f709fe276eb7c8 Mon Sep 17 00:00:00 2001
From: Andreas Kling <kling@serenityos.org>
Date: Sun, 5 Sep 2021 18:22:18 +0200
Subject: [PATCH] Kernel: Use TRY() in sys$sethostname()

---
 Kernel/Syscalls/hostname.cpp | 6 ++----
 1 file changed, 2 insertions(+), 4 deletions(-)

diff --git a/Kernel/Syscalls/hostname.cpp b/Kernel/Syscalls/hostname.cpp
index 69f2132e3f..7dd259522a 100644
--- a/Kernel/Syscalls/hostname.cpp
+++ b/Kernel/Syscalls/hostname.cpp
@@ -30,12 +30,10 @@ KResultOr<FlatPtr> Process::sys$sethostname(Userspace<const char*> buffer, size_
         return EPERM;
     if (length > 64)
         return ENAMETOOLONG;
+    auto new_name = TRY(try_copy_kstring_from_user(buffer, length));
     return hostname().with_exclusive([&](auto& name) -> KResultOr<FlatPtr> {
-        auto name_or_error = try_copy_kstring_from_user(buffer, length);
-        if (name_or_error.is_error())
-            return name_or_error.error();
         // FIXME: Use KString instead of String here.
-        name = name_or_error.value()->view();
+        name = new_name->view();
         return 0;
     });
 }