From eb857e45dc1cb6aa6413c2ef2d848a7ddf3eb2f1 Mon Sep 17 00:00:00 2001
From: Sam Atkins <atkinssj@serenityos.org>
Date: Sat, 2 Apr 2022 13:49:58 +0100
Subject: [PATCH] HackStudio: Use Core::System::exec()

---
 Userland/DevTools/HackStudio/TerminalWrapper.cpp | 15 +++------------
 1 file changed, 3 insertions(+), 12 deletions(-)

diff --git a/Userland/DevTools/HackStudio/TerminalWrapper.cpp b/Userland/DevTools/HackStudio/TerminalWrapper.cpp
index 1c1ee04c0b..b44f26fd16 100644
--- a/Userland/DevTools/HackStudio/TerminalWrapper.cpp
+++ b/Userland/DevTools/HackStudio/TerminalWrapper.cpp
@@ -60,18 +60,9 @@ ErrorOr<void> TerminalWrapper::run_command(String const& command, Optional<Strin
 
     TRY(setup_slave_pseudoterminal(ptm_fd));
 
-    auto parts = command.split(' ');
-    VERIFY(!parts.is_empty());
-    char const** args = (char const**)calloc(parts.size() + 1, sizeof(char const*));
-    for (size_t i = 0; i < parts.size(); i++) {
-        args[i] = parts[i].characters();
-    }
-
-    auto rc = execvp(args[0], const_cast<char**>(args));
-    if (rc < 0) {
-        perror("execve");
-        exit(1);
-    }
+    auto args = command.split_view(' ');
+    VERIFY(!args.is_empty());
+    TRY(Core::System::exec(args[0], args, Core::System::SearchInPath::Yes));
     VERIFY_NOT_REACHED();
 }