Everywhere: Replace some uses of fork/exec with posix_spawn

It's less code, and it's potentially more efficient once
posix_spawn is a real syscall.

1 files changed, 5 insertions, 10 deletions

diff --git a/Applications/KeyboardSettings/main.cpp b/Applications/KeyboardSettings/main.cpp
index c2a36c9344..0cb4aaad5a 100644
--- a/Applications/KeyboardSettings/main.cpp
+++ b/Applications/KeyboardSettings/main.cpp
@@ -40,6 +40,7 @@
 #include <LibGUI/MessageBox.h>
 #include <LibGUI/WindowServerConnection.h>
 #include <LibKeyboard/CharacterMap.h>
+#include <spawn.h>
 
 int main(int argc, char** argv)
 {
@@ -123,18 +124,12 @@ int main(int argc, char** argv)
             GUI::MessageBox::show("Please select character mapping file.", "Keyboard settings", GUI::MessageBox::Type::Error, GUI::MessageBox::InputType::OK, window);
             return;
         }
-
-        pid_t child_pid = fork();
-        if (child_pid < 0) {
-            perror("fork");
+        pid_t child_pid;
+        const char* argv[] = { "/bin/keymap", character_map_file.characters(), nullptr };
+        if ((errno = posix_spawn(&child_pid, "/bin/keymap", nullptr, nullptr, const_cast<char**>(argv), environ))) {
+            perror("posix_spawn");
             exit(1);
         }
-        if (child_pid == 0) {
-            if (execl("/bin/keymap", "/bin/keymap", character_map_file.characters(), nullptr) < 0) {
-                perror("execl");
-                exit(1);
-            }
-        }
         if (quit)
             app.quit();
     };