TraceScreen: fix unnecessary copying parent memory page table

posix_spawn() is a more lightweight and modern alternative to fork()/exec(), which are used inside popen(). The main advantage of posix_spawn is that it can create a new process without completely duplicating the address space of the parent process.

References:
- https://blog.famzah.net/2018/12/19/posix_spawn-performance-benchmarks-and-usage-examples/
- https://lobste.rs/s/smbsd5/fork_road
- https://www.reddit.com/r/C_Programming/comments/1lvdhp2/fork_vs_posix_spawn/

Author: GermanAizek

TraceScreen.c

@@ -13,6 +13,7 @@ in the source distribution for its full text.
 #include <errno.h>
 #include <fcntl.h>
 #include <signal.h>
+#include <spawn.h>
 #include <stdbool.h>
 #include <stdio.h>
 #include <stdlib.h>
@@ -78,42 +79,36 @@ bool TraceScreen_forkTracer(TraceScreen* this) {
    if (fcntl(fdpair[1], F_SETFL, O_NONBLOCK) < 0)
       goto err;
 
-   pid_t child = fork();
-   if (child == -1)
+   pid_t child;
+   posix_spawnattr_t attr;
+   posix_spawnattr_init(&attr);
+   posix_spawn_file_actions_t fa;
+   posix_spawn_file_actions_init(&fa);
+   posix_spawn_file_actions_addclose(&fa, fdpair[0]);
+   posix_spawn_file_actions_adddup2(&fa, fdpair[1], STDOUT_FILENO);
+   posix_spawn_file_actions_adddup2(&fa, fdpair[1], STDERR_FILENO);
+   posix_spawn_file_actions_addclose(&fa, fdpair[1]);
+
+   char buffer[32] = {0};
+   xSnprintf(buffer, sizeof(buffer), "%d", Process_getPid(this->super.process));
+
+#if defined(HTOP_FREEBSD) || defined(HTOP_OPENBSD) || defined(HTOP_NETBSD) || defined(HTOP_DRAGONFLYBSD) || defined(HTOP_SOLARIS)
+   char* const args[] = { (char*)"truss", (char*)"-s", (char*)"512", (char*)"-p", buffer, NULL };
+#elif defined(HTOP_LINUX)
+   char* const args[] = { (char*)"strace", (char*)"-T", (char*)"-tt", (char*)"-s", (char*)"512", (char*)"-p", buffer, NULL };
+#else
+   char* const args[] = { NULL };
+#endif
+
+   if (!args[0] || posix_spawn(&child, args[0], &fa, &attr, args, NULL) != 0) {
+      posix_spawnattr_destroy(&attr);
+      posix_spawn_file_actions_destroy(&fa);
       goto err;
-
-   if (child == 0) {
-      close(fdpair[0]);
-
-      dup2(fdpair[1], STDOUT_FILENO);
-      dup2(fdpair[1], STDERR_FILENO);
-      close(fdpair[1]);
-
-      char buffer[32] = {0};
-      xSnprintf(buffer, sizeof(buffer), "%d", Process_getPid(this->super.process));
-
-      #if defined(HTOP_FREEBSD) || defined(HTOP_OPENBSD) || defined(HTOP_NETBSD) || defined(HTOP_DRAGONFLYBSD) || defined(HTOP_SOLARIS)
-         // Use of NULL in variadic functions must have a pointer cast.
-         // The NULL constant is not required by standard to have a pointer type.
-         execlp("truss", "truss", "-s", "512", "-p", buffer, (void*)NULL);
-
-         // Should never reach here, unless execlp fails ...
-         const char* message = "Could not execute 'truss'. Please make sure it is available in your $PATH.";
-         (void)! write(STDERR_FILENO, message, strlen(message));
-      #elif defined(HTOP_LINUX)
-         execlp("strace", "strace", "-T", "-tt", "-s", "512", "-p", buffer, (void*)NULL);
-
-         // Should never reach here, unless execlp fails ...
-         const char* message = "Could not execute 'strace'. Please make sure it is available in your $PATH.";
-         (void)! write(STDERR_FILENO, message, strlen(message));
-      #else // HTOP_DARWIN, HTOP_PCP == HTOP_UNSUPPORTED
-         const char* message = "Tracing unavailable on not supported system.";
-         (void)! write(STDERR_FILENO, message, strlen(message));
-      #endif
-
-      exit(127);
    }
 
+   posix_spawnattr_destroy(&attr);
+   posix_spawn_file_actions_destroy(&fa);
+
    FILE* fp = fdopen(fdpair[0], "r");
    if (!fp)
       goto err;