]> git.ipfire.org Git - people/ms/pakfire.git/blobdiff - src/libpakfire/jail.c
jail: Implement PTY forwarding
[people/ms/pakfire.git] / src / libpakfire / jail.c
index 40ea2348acd3f307d95fa89ab8642bd558628bb4..5f3884445c04117f8e3365b15983f6d536e393ed 100644 (file)
 #include <sys/personality.h>
 #include <sys/prctl.h>
 #include <sys/resource.h>
-#include <sys/signalfd.h>
 #include <sys/timerfd.h>
 #include <sys/types.h>
 #include <sys/wait.h>
+#include <termios.h>
 
 // libnl3
 #include <net/if.h>
@@ -56,6 +56,7 @@
 #include <pakfire/logging.h>
 #include <pakfire/mount.h>
 #include <pakfire/pakfire.h>
+#include <pakfire/path.h>
 #include <pakfire/private.h>
 #include <pakfire/pwd.h>
 #include <pakfire/string.h>
@@ -88,6 +89,7 @@ struct pakfire_jail_mountpoint {
 };
 
 struct pakfire_jail {
+       struct pakfire_ctx* ctx;
        struct pakfire* pakfire;
        int nrefs;
 
@@ -131,6 +133,9 @@ struct pakfire_jail_exec {
        pid_t pid;
        int pidfd;
 
+       // Socket to pass FDs
+       int socket[2];
+
        // Process status (from waitid)
        siginfo_t status;
 
@@ -139,14 +144,12 @@ struct pakfire_jail_exec {
 
        // Log pipes
        struct pakfire_jail_pipes {
-               int stdin[2];
-               int stdout[2];
-               int stderr[2];
-
                // Logging
                int log_INFO[2];
                int log_ERROR[2];
+#ifdef ENABLE_DEBUG
                int log_DEBUG[2];
+#endif /* ENABLE_DEBUG */
        } pipes;
 
        // Communicate
@@ -164,11 +167,41 @@ struct pakfire_jail_exec {
                // Logging
                struct pakfire_log_buffer log_INFO;
                struct pakfire_log_buffer log_ERROR;
+#ifdef ENABLE_DEBUG
                struct pakfire_log_buffer log_DEBUG;
+#endif /* ENABLE_DEBUG */
        } buffers;
 
        struct pakfire_cgroup* cgroup;
        struct pakfire_cgroup_stats cgroup_stats;
+
+       // PTY
+       struct pakfire_jail_pty {
+               // The path to the console
+               char console[PATH_MAX];
+
+               // The master fd
+               struct pakfire_jail_pty_master {
+                       int fd;
+
+                       enum pakfire_jail_pty_flags {
+                               PAKFIRE_JAIL_PTY_READY_TO_READ  = (1 << 0),
+                               PAKFIRE_JAIL_PTY_READY_TO_WRITE = (1 << 1),
+                       } flags;
+               } master;
+
+               // Standard Input
+               struct pakfire_jail_pty_stdio {
+                       int fd;
+                       struct pakfire_log_buffer buffer;
+                       struct termios attrs;
+                       int fdflags;
+                       enum pakfire_jail_pty_flags flags;
+               } stdin;
+
+               // Standard Output
+               struct pakfire_jail_pty_stdio stdout;
+       } pty;
 };
 
 static int clone3(struct clone_args* args, size_t size) {
@@ -197,8 +230,10 @@ static void pakfire_jail_free(struct pakfire_jail* jail) {
 
        if (jail->cgroup)
                pakfire_cgroup_unref(jail->cgroup);
-
-       pakfire_unref(jail->pakfire);
+       if (jail->pakfire)
+               pakfire_unref(jail->pakfire);
+       if (jail->ctx)
+               pakfire_ctx_unref(jail->ctx);
        free(jail);
 }
 
@@ -268,6 +303,9 @@ PAKFIRE_EXPORT int pakfire_jail_create(struct pakfire_jail** jail, struct pakfir
        if (!j)
                return 1;
 
+       // Reference context
+       j->ctx = pakfire_ctx(pakfire);
+
        // Reference Pakfire
        j->pakfire = pakfire_ref(pakfire);
 
@@ -481,7 +519,7 @@ PAKFIRE_EXPORT int pakfire_jail_set_timeout(
        jail->timeout.it_value.tv_sec = timeout;
 
        if (timeout > 0)
-               DEBUG(jail->pakfire, "Timeout set to %d second(s)\n", timeout);
+               DEBUG(jail->pakfire, "Timeout set to %u second(s)\n", timeout);
        else
                DEBUG(jail->pakfire, "Timeout disabled\n");
 
@@ -512,44 +550,18 @@ static int pakfire_jail_create_timer(struct pakfire_jail* jail) {
        return fd;
 
 ERROR:
-       if (fd > 0)
+       if (fd >= 0)
                close(fd);
 
        return -1;
 }
 
-// Signals
-
-static int pakfire_jail_handle_signals(struct pakfire_jail* jail) {
-       sigset_t mask;
-       int r;
-
-       sigemptyset(&mask);
-       sigaddset(&mask, SIGINT);
-
-       // Block signals
-       r = sigprocmask(SIG_BLOCK, &mask, NULL);
-       if (r < 0) {
-               ERROR(jail->pakfire, "Failed to block signals: %m\n");
-               return r;
-       }
-
-       // Create a file descriptor
-       r = signalfd(-1, &mask, SFD_NONBLOCK|SFD_CLOEXEC);
-       if (r < 0) {
-               ERROR(jail->pakfire, "Failed to create signalfd: %m\n");
-               return r;
-       }
-
-       return r;
-}
-
 /*
        This function replaces any logging in the child process.
 
        All log messages will be sent to the parent process through their respective pipes.
 */
-static void pakfire_jail_log(void* data, int priority, const char* file,
+static void pakfire_jail_log_redirect(void* data, int priority, const char* file,
                int line, const char* fn, const char* format, va_list args) {
        struct pakfire_jail_pipes* pipes = (struct pakfire_jail_pipes*)data;
        int fd;
@@ -575,7 +587,7 @@ static void pakfire_jail_log(void* data, int priority, const char* file,
        }
 
        // Send the log message
-       if (fd)
+       if (fd >= 0)
                vdprintf(fd, format, args);
 }
 
@@ -653,6 +665,7 @@ static int pakfire_jail_handle_log(struct pakfire_jail* jail,
        return 0;
 }
 
+#if 0
 static int pakfire_jail_stream_stdin(struct pakfire_jail* jail,
                struct pakfire_jail_exec* ctx, const int fd) {
        int r;
@@ -664,7 +677,7 @@ static int pakfire_jail_stream_stdin(struct pakfire_jail* jail,
        }
 
        // Skip if the writing pipe has already been closed
-       if (!ctx->pipes.stdin[1])
+       if (ctx->pipes.stdin[1] < 0)
                return 0;
 
        DEBUG(jail->pakfire, "Streaming standard input...\n");
@@ -682,7 +695,7 @@ static int pakfire_jail_stream_stdin(struct pakfire_jail* jail,
                close(fd);
 
                // Reset the file-descriptor so it won't be closed again later
-               ctx->pipes.stdin[1] = 0;
+               ctx->pipes.stdin[1] = -1;
 
                // Report success
                r = 0;
@@ -690,6 +703,68 @@ static int pakfire_jail_stream_stdin(struct pakfire_jail* jail,
 
        return r;
 }
+#endif
+
+static int pakfire_jail_recv_fd(struct pakfire_jail* jail, int socket, int* fd) {
+       const size_t payload_length = sizeof(fd);
+       char buffer[CMSG_SPACE(payload_length)];
+       int r;
+
+       struct msghdr msg = {
+               .msg_control    = buffer,
+               .msg_controllen = sizeof(buffer),
+       };
+
+       // Receive the message
+       r = recvmsg(socket, &msg, 0);
+       if (r) {
+               CTX_ERROR(jail->ctx, "Could not receive file descriptor: %s\n", strerror(errno));
+               return -errno;
+       }
+
+       // Fetch the payload
+       struct cmsghdr* cmsg = CMSG_FIRSTHDR(&msg);
+       if (!cmsg)
+               return -EBADMSG;
+
+       *fd = *((int*)CMSG_DATA(cmsg));
+
+       CTX_DEBUG(jail->ctx, "Received fd %d from socket %d\n", *fd, socket);
+
+       return 0;
+}
+
+static int pakfire_jail_send_fd(struct pakfire_jail* jail, int socket, int fd) {
+       const size_t payload_length = sizeof(fd);
+       char buffer[CMSG_SPACE(payload_length)];
+       int r;
+
+       CTX_DEBUG(jail->ctx, "Sending fd %d to socket %d\n", fd, socket);
+
+       // Header
+       struct msghdr msg = {
+               .msg_control    = buffer,
+               .msg_controllen = sizeof(buffer),
+       };
+
+       // Payload
+       struct cmsghdr* cmsg = CMSG_FIRSTHDR(&msg);
+       cmsg->cmsg_level = SOL_SOCKET;
+       cmsg->cmsg_type  = SCM_RIGHTS;
+       cmsg->cmsg_len   = CMSG_LEN(payload_length);
+
+       // Set payload
+       *((int*)CMSG_DATA(cmsg)) = fd;
+
+       // Send the message
+       r = sendmsg(socket, &msg, 0);
+       if (r) {
+               CTX_ERROR(jail->ctx, "Could not send file descriptor: %s\n", strerror(errno));
+               return -errno;
+       }
+
+       return 0;
+}
 
 static int pakfire_jail_setup_pipe(struct pakfire_jail* jail, int (*fds)[2], const int flags) {
        int r = pipe2(*fds, flags);
@@ -703,7 +778,7 @@ static int pakfire_jail_setup_pipe(struct pakfire_jail* jail, int (*fds)[2], con
 
 static void pakfire_jail_close_pipe(struct pakfire_jail* jail, int fds[2]) {
        for (unsigned int i = 0; i < 2; i++)
-               if (fds[i])
+               if (fds[i] >= 0)
                        close(fds[i]);
 }
 
@@ -717,13 +792,16 @@ static int pakfire_jail_get_pipe_to_read(struct pakfire_jail* jail, int (*fds)[2
        int* fd_write = &(*fds)[1];
 
        // Close the write end of the pipe
-       if (*fd_write) {
+       if (*fd_write >= 0) {
                close(*fd_write);
                *fd_write = -1;
        }
 
        // Return the read end
-       return *fd_read;
+       if (*fd_read >= 0)
+               return *fd_read;
+
+       return -1;
 }
 
 static int pakfire_jail_get_pipe_to_write(struct pakfire_jail* jail, int (*fds)[2]) {
@@ -732,43 +810,358 @@ static int pakfire_jail_get_pipe_to_write(struct pakfire_jail* jail, int (*fds)[
        int* fd_write = &(*fds)[1];
 
        // Close the read end of the pipe
-       if (*fd_read) {
+       if (*fd_read >= 0) {
                close(*fd_read);
                *fd_read = -1;
        }
 
        // Return the write end
-       return *fd_write;
+       if (*fd_write >= 0)
+               return *fd_write;
+
+       return -1;
+}
+
+static int pakfire_jail_log(struct pakfire* pakfire, void* data, int priority,
+               const char* line, const size_t length) {
+       // Pass everything to the parent logger
+       pakfire_log_condition(pakfire, priority, 0, "%.*s", (int)length, line);
+
+       return 0;
+}
+
+static int pakfire_jail_epoll_add_fd(struct pakfire_jail* jail, int epollfd, int fd, int events) {
+       struct epoll_event event = {
+               .events = events|EPOLLHUP,
+               .data   = {
+                       .fd = fd,
+               },
+       };
+       int r;
+
+       // Read flags
+       int flags = fcntl(fd, F_GETFL, 0);
+
+       // Set modified flags
+       r  = fcntl(fd, F_SETFL, flags|O_NONBLOCK);
+       if (r < 0) {
+               CTX_ERROR(jail->ctx, "Could not set file descriptor %d into non-blocking mode: %s\n",
+                       fd, strerror(errno));
+               return -errno;
+       }
+
+       // Add the file descriptor to the loop
+       r = epoll_ctl(epollfd, EPOLL_CTL_ADD, fd, &event);
+       if (r < 0) {
+               ERROR(jail->pakfire, "Could not add file descriptor %d to epoll(): %s\n",
+                       fd, strerror(errno));
+               return -errno;
+       }
+
+       return 0;
+}
+
+// PTY Forwarding
+
+static int pakfire_jail_enable_raw_mode(struct pakfire_jail* jail,
+               struct pakfire_jail_pty_stdio* stdio) {
+       struct termios raw_attrs;
+       int r;
+
+       // Store flags
+       stdio->fdflags = fcntl(stdio->fd, F_GETFL);
+       if (stdio->fdflags < 0) {
+               CTX_ERROR(jail->ctx, "Could not fetch flags from fd %d: %s\n",
+                       stdio->fd, strerror(errno));
+               return -errno;
+       }
+
+       // Fetch all attributes
+       r = tcgetattr(stdio->fd, &stdio->attrs);
+       if (r) {
+               CTX_ERROR(jail->ctx, "Could not fetch terminal attributes from fd %d: %s\n",
+                       stdio->fd, strerror(errno));
+               return -errno;
+       }
+
+       // Copy all attributes
+       raw_attrs = stdio->attrs;
+
+       // Make it RAW
+       cfmakeraw(&raw_attrs);
+
+       switch (stdio->fd) {
+               case STDIN_FILENO:
+                       raw_attrs.c_oflag = stdio->attrs.c_oflag;
+                       break;
+
+               case STDOUT_FILENO:
+                       raw_attrs.c_iflag = stdio->attrs.c_iflag;
+                       raw_attrs.c_lflag = stdio->attrs.c_lflag;
+                       break;
+       }
+
+       // Restore the attributes
+       r = tcsetattr(stdio->fd, TCSANOW, &raw_attrs);
+       if (r) {
+               CTX_ERROR(jail->ctx, "Could not restore terminal attributes for fd %d: %s\n",
+                       stdio->fd, strerror(errno));
+               return -errno;
+       }
+
+       return 0;
+}
+
+static int pakfire_jail_restore_attrs(struct pakfire_jail* jail,
+               const struct pakfire_jail_pty_stdio* stdio) {
+       int r;
+
+       // Restore the flags
+       r = fcntl(stdio->fd, F_SETFL, stdio->fdflags);
+       if (r < 0) {
+               CTX_ERROR(jail->ctx, "Could not set flags for file descriptor %d: %s\n",
+                       stdio->fd, strerror(errno));
+               return -errno;
+       }
+
+       // Restore the attributes
+       r = tcsetattr(stdio->fd, TCSANOW, &stdio->attrs);
+       if (r) {
+               CTX_ERROR(jail->ctx, "Could not restore terminal attributes for %d, ignoring: %s\n",
+                       stdio->fd, strerror(errno));
+               return -errno;
+       }
+
+       return 0;
+}
+
+static int pakfire_jail_setup_pty_forwarding(struct pakfire_jail* jail,
+               struct pakfire_jail_exec* ctx, const int epollfd, const int fd) {
+       struct winsize size;
+       int r;
+
+       CTX_DEBUG(jail->ctx, "Setting up PTY forwarding on fd %d\n", fd);
+
+       // Store the file descriptor
+       ctx->pty.master.fd = fd;
+
+       // Configure stdin/stdout
+       ctx->pty.stdin.fd  = STDIN_FILENO;
+       ctx->pty.stdout.fd = STDOUT_FILENO;
+
+       // Fetch dimensions
+       r = ioctl(ctx->pty.stdout.fd, TIOCGWINSZ, &size);
+       if (r) {
+               CTX_ERROR(jail->ctx, "Failed to determine terminal dimensions: %s\n", strerror(errno));
+               return -errno;
+       }
+
+       // Set dimensions
+       r = ioctl(ctx->pty.master.fd, TIOCSWINSZ, &size);
+       if (r) {
+               CTX_ERROR(jail->ctx, "Failed setting dimensions: %s\n", strerror(errno));
+               return -errno;
+       }
+
+       // Enable RAW mode on standard input
+       r = pakfire_jail_enable_raw_mode(jail, &ctx->pty.stdin);
+       if (r)
+               return r;
+
+       // Enable RAW mode on standard output
+       r = pakfire_jail_enable_raw_mode(jail, &ctx->pty.stdout);
+       if (r)
+               return r;
+
+       // Add the master to the event loop
+       r = pakfire_jail_epoll_add_fd(jail, epollfd, ctx->pty.master.fd, EPOLLIN|EPOLLOUT|EPOLLET);
+       if (r)
+               return r;
+
+       // Add standard input to the event loop
+       r = pakfire_jail_epoll_add_fd(jail, epollfd, ctx->pty.stdin.fd, EPOLLIN|EPOLLET);
+       if (r)
+               return r;
+
+       // Add standard output to the event loop
+       r = pakfire_jail_epoll_add_fd(jail, epollfd, ctx->pty.stdout.fd, EPOLLOUT|EPOLLET);
+       if (r)
+               return r;
+
+       return 0;
+}
+
+static int pakfire_jail_fill_buffer(struct pakfire_jail* jail, int fd, struct pakfire_log_buffer* buffer) {
+       int r;
+
+       // Skip this if there is not space left in the buffer
+       if (buffer->used >= sizeof(buffer->data))
+               return 0;
+
+       // Fill the buffer
+       r = read(fd, buffer->data + buffer->used, sizeof(buffer->data) - buffer->used);
+
+       // Handle errors
+       if (r < 0) {
+               switch (errno) {
+                       case EAGAIN:
+                       case EIO:
+                               break;
+
+                       default:
+                               return -errno;
+               }
+
+       // EOF
+       } else if (r == 0) {
+               // XXX What to do here?
+
+       // Successful read
+       } else {
+               buffer->used += r;
+       }
+
+       return 0;
+}
+
+static int pakfire_jail_drain_buffer(struct pakfire_jail* jail, int fd, struct pakfire_log_buffer* buffer) {
+       int r;
+
+       // Nothing to do if the buffer is empty
+       if (!buffer->used)
+               return 0;
+
+       // Drain the buffer
+       r = write(fd, buffer->data, buffer->used);
+
+       // Handle errors
+       if (r < 0) {
+               switch (errno) {
+                       case EAGAIN:
+                       case EIO:
+                               break;
+
+                       default:
+                               return -errno;
+               }
+
+       // Successful write
+       } else {
+               memmove(buffer->data, buffer->data + r, buffer->used - r);
+
+               buffer->used -= r;
+       }
+
+       return 0;
+}
+
+static int pakfire_jail_forward_pty(struct pakfire_jail* jail, struct pakfire_jail_exec* ctx) {
+       int r;
+
+       // Read from standard input
+       if (ctx->pty.stdin.flags & PAKFIRE_JAIL_PTY_READY_TO_READ) {
+               r = pakfire_jail_fill_buffer(jail, ctx->pty.stdin.fd, &ctx->pty.stdin.buffer);
+               if (r) {
+                       CTX_ERROR(jail->ctx, "Failed reading from standard input: %s\n", strerror(-r));
+                       return r;
+               }
+
+               // We are done reading for now
+               ctx->pty.stdin.flags &= ~PAKFIRE_JAIL_PTY_READY_TO_READ;
+
+               // But we may have data to write
+               if (ctx->pty.stdin.buffer.used)
+                       ctx->pty.master.flags |= PAKFIRE_JAIL_PTY_READY_TO_WRITE;
+       }
+
+       // Write to the master
+       if (ctx->pty.master.flags & PAKFIRE_JAIL_PTY_READY_TO_WRITE) {
+               r = pakfire_jail_drain_buffer(jail, ctx->pty.master.fd, &ctx->pty.stdin.buffer);
+               if (r) {
+                       CTX_ERROR(jail->ctx, "Failed writing to the PTY: %s\n", strerror(-r));
+                       return r;
+               }
+
+               // We are done writing for now
+               ctx->pty.master.flags &= ~PAKFIRE_JAIL_PTY_READY_TO_WRITE;
+       }
+
+       // Read from the master
+       if (ctx->pty.master.flags & PAKFIRE_JAIL_PTY_READY_TO_READ) {
+               r = pakfire_jail_fill_buffer(jail, ctx->pty.master.fd, &ctx->pty.stdout.buffer);
+               if (r) {
+                       CTX_ERROR(jail->ctx, "Failed reading from the PTY: %s\n", strerror(-r));
+                       return r;
+               }
+
+               // We are done reading for now
+               ctx->pty.master.flags &= ~PAKFIRE_JAIL_PTY_READY_TO_READ;
+
+               // But we may have data to write
+               if (ctx->pty.stdout.buffer.used)
+                       ctx->pty.stdout.flags |= PAKFIRE_JAIL_PTY_READY_TO_WRITE;
+       }
+
+       // Write to standard output
+       if (ctx->pty.stdout.flags & PAKFIRE_JAIL_PTY_READY_TO_WRITE) {
+               r = pakfire_jail_drain_buffer(jail, ctx->pty.stdout.fd, &ctx->pty.stdout.buffer);
+               if (r) {
+                       CTX_ERROR(jail->ctx, "Failed writing to standard output: %s\n", strerror(-r));
+                       return r;
+               }
+
+               // We are done writing for now
+               ctx->pty.stdout.flags &= ~PAKFIRE_JAIL_PTY_READY_TO_WRITE;
+       }
+
+       return 0;
 }
 
 static int pakfire_jail_wait(struct pakfire_jail* jail, struct pakfire_jail_exec* ctx) {
        int epollfd = -1;
-       struct epoll_event ev;
        struct epoll_event events[EPOLL_MAX_EVENTS];
-       struct signalfd_siginfo siginfo;
        char garbage[8];
        int r = 0;
 
        // Fetch file descriptors from context
-       const int stdin = pakfire_jail_get_pipe_to_write(jail, &ctx->pipes.stdin);
-       const int stdout = pakfire_jail_get_pipe_to_read(jail, &ctx->pipes.stdout);
-       const int stderr = pakfire_jail_get_pipe_to_read(jail, &ctx->pipes.stderr);
        const int pidfd  = ctx->pidfd;
 
+       // Fetch the UNIX domain socket
+       const int socket_recv = pakfire_jail_get_pipe_to_read(jail, &ctx->socket);
+
        // Timer
        const int timerfd = pakfire_jail_create_timer(jail);
 
        // Logging
        const int log_INFO  = pakfire_jail_get_pipe_to_read(jail, &ctx->pipes.log_INFO);
        const int log_ERROR = pakfire_jail_get_pipe_to_read(jail, &ctx->pipes.log_ERROR);
+#ifdef ENABLE_DEBUG
        const int log_DEBUG = pakfire_jail_get_pipe_to_read(jail, &ctx->pipes.log_DEBUG);
-
-       // Signals
-       const int signalfd = pakfire_jail_handle_signals(jail);
+#endif /* ENABLE_DEBUG */
 
        // Make a list of all file descriptors we are interested in
-       const int fds[] = {
-               stdin, stdout, stderr, pidfd, timerfd, signalfd, log_INFO, log_ERROR, log_DEBUG,
+       const struct pakfire_wait_fds {
+               const int fd;
+               const int events;
+       } fds[] = {
+               // Timer
+               { timerfd, EPOLLIN },
+
+               // Child Process
+               { ctx->pidfd, EPOLLIN },
+
+               // Log Pipes
+               { log_INFO, EPOLLIN },
+               { log_ERROR, EPOLLIN },
+#ifdef ENABLE_DEBUG
+               { log_DEBUG, EPOLLIN },
+#endif /* ENABLE_DEBUG */
+
+               // UNIX Domain Socket
+               { socket_recv, EPOLLIN },
+
+               // Sentinel
+               { -1, 0 },
        };
 
        // Setup epoll
@@ -780,38 +1173,15 @@ static int pakfire_jail_wait(struct pakfire_jail* jail, struct pakfire_jail_exec
        }
 
        // Turn file descriptors into non-blocking mode and add them to epoll()
-       for (unsigned int i = 0; i < sizeof(fds) / sizeof(*fds); i++) {
-               int fd = fds[i];
-
+       for (const struct pakfire_wait_fds* fd = fds; fd->events; fd++) {
                // Skip fds which were not initialized
-               if (fd < 0)
+               if (fd->fd < 0)
                        continue;
 
-               ev.events = EPOLLHUP;
-
-               if (fd == stdin)
-                       ev.events |= EPOLLOUT;
-               else
-                       ev.events |= EPOLLIN;
-
-               // Read flags
-               int flags = fcntl(fd, F_GETFL, 0);
-
-               // Set modified flags
-               if (fcntl(fd, F_SETFL, flags|O_NONBLOCK) < 0) {
-                       ERROR(jail->pakfire,
-                               "Could not set file descriptor %d into non-blocking mode: %m\n", fd);
-                       r = 1;
-                       goto ERROR;
-               }
-
-               ev.data.fd = fd;
-
-               if (epoll_ctl(epollfd, EPOLL_CTL_ADD, fd, &ev) < 0) {
-                       ERROR(jail->pakfire, "Could not add file descriptor %d to epoll(): %m\n", fd);
-                       r = 1;
+               // Add the FD to the event loop
+               r = pakfire_jail_epoll_add_fd(jail, epollfd, fd->fd, fd->events);
+               if (r)
                        goto ERROR;
-               }
        }
 
        int ended = 0;
@@ -839,6 +1209,46 @@ static int pakfire_jail_wait(struct pakfire_jail* jail, struct pakfire_jail_exec
                        void* data = NULL;
                        int priority;
 
+                       // Handle PTY forwarding events
+                       if (ctx->pty.master.fd == fd) {
+                               if (e & (EPOLLIN|EPOLLHUP))
+                                       ctx->pty.master.flags |= PAKFIRE_JAIL_PTY_READY_TO_READ;
+
+                               if (e & (EPOLLOUT|EPOLLHUP))
+                                       ctx->pty.master.flags |= PAKFIRE_JAIL_PTY_READY_TO_WRITE;
+
+                               // Perform the work
+                               r = pakfire_jail_forward_pty(jail, ctx);
+                               if (r) {
+                                       CTX_ERROR(jail->ctx, "Failed forwarding the PTY: %s\n", strerror(-r));
+                                       goto ERROR;
+                               }
+
+                       // Handle standard input
+                       } else if (ctx->pty.stdin.fd == fd) {
+                               if (e & (EPOLLIN|EPOLLHUP))
+                                       ctx->pty.stdin.flags |= PAKFIRE_JAIL_PTY_READY_TO_READ;
+
+                               // Perform the work
+                               r = pakfire_jail_forward_pty(jail, ctx);
+                               if (r) {
+                                       CTX_ERROR(jail->ctx, "Failed forwarding the PTY: %s\n", strerror(-r));
+                                       goto ERROR;
+                               }
+
+                       // Handle standard output
+                       } else if (ctx->pty.stdout.fd == fd) {
+                               if (e & (EPOLLOUT|EPOLLHUP))
+                                       ctx->pty.stdout.flags |= PAKFIRE_JAIL_PTY_READY_TO_WRITE;
+
+                               // Perform the work
+                               r = pakfire_jail_forward_pty(jail, ctx);
+                               if (r) {
+                                       CTX_ERROR(jail->ctx, "Failed forwarding the PTY: %s\n", strerror(-r));
+                                       goto ERROR;
+                               }
+                       }
+
                        // Check if there is any data to be read
                        if (e & EPOLLIN) {
                                // Handle any changes to the PIDFD
@@ -879,37 +1289,6 @@ static int pakfire_jail_wait(struct pakfire_jail* jail, struct pakfire_jail_exec
                                                }
                                        }
 
-                                       // There is nothing else to do
-                                       continue;
-
-                               // Handle signals
-                               } else if (fd == signalfd) {
-                                       // Read the signal
-                                       r = read(signalfd, &siginfo, sizeof(siginfo));
-                                       if (r < 1) {
-                                               ERROR(jail->pakfire, "Could not read signal: %m\n");
-                                               goto ERROR;
-                                       }
-
-                                       DEBUG(jail->pakfire, "Received signal %d\n", siginfo.ssi_signo);
-
-                                       // Handle signals
-                                       switch (siginfo.ssi_signo) {
-                                               // Pass SIGINT down to the child process
-                                               case SIGINT:
-                                                       r = pidfd_send_signal(pidfd, siginfo.ssi_signo, NULL, 0);
-                                                       if (r) {
-                                                               ERROR(jail->pakfire, "Could not send signal to process: %m\n");
-                                                               goto ERROR;
-                                                       }
-                                                       break;
-
-                                               default:
-                                                       ERROR(jail->pakfire, "Received unhandled signal %d\n",
-                                                               siginfo.ssi_signo);
-                                                       break;
-                                       }
-
                                        // Don't fall through to log processing
                                        continue;
 
@@ -918,49 +1297,40 @@ static int pakfire_jail_wait(struct pakfire_jail* jail, struct pakfire_jail_exec
                                        buffer = &ctx->buffers.log_INFO;
                                        priority = LOG_INFO;
 
-                                       callback = jail->callbacks.log;
-                                       data     = jail->callbacks.log_data;
+                                       callback = pakfire_jail_log;
 
                                } else if (fd == log_ERROR) {
                                        buffer = &ctx->buffers.log_ERROR;
                                        priority = LOG_ERR;
 
-                                       callback = jail->callbacks.log;
-                                       data     = jail->callbacks.log_data;
+                                       callback = pakfire_jail_log;
 
+#ifdef ENABLE_DEBUG
                                } else if (fd == log_DEBUG) {
                                        buffer = &ctx->buffers.log_DEBUG;
                                        priority = LOG_DEBUG;
 
-                                       callback = jail->callbacks.log;
-                                       data     = jail->callbacks.log_data;
+                                       callback = pakfire_jail_log;
+#endif /* ENABLE_DEBUG */
 
-                               // Handle anything from the log pipes
-                               } else if (fd == stdout) {
-                                       buffer = &ctx->buffers.stdout;
-                                       priority = LOG_INFO;
+                               // Handle socket messages
+                               } else if (fd == socket_recv) {
+                                       // Receive the passed FD
+                                       r = pakfire_jail_recv_fd(jail, socket_recv, &fd);
+                                       if (r)
+                                               goto ERROR;
 
-                                       // Send any output to the default logger if no callback is set
-                                       if (ctx->communicate.out) {
-                                               callback = ctx->communicate.out;
-                                               data     = ctx->communicate.data;
-                                       } else {
-                                               callback = jail->callbacks.log;
-                                               data     = jail->callbacks.log_data;
+                                       // Setup PTY forwarding
+                                       if (ctx->pty.master.fd < 0) {
+                                               r = pakfire_jail_setup_pty_forwarding(jail, ctx, epollfd, fd);
+                                               if (r) {
+                                                       CTX_ERROR(jail->ctx, "Failed setting up PTY forwarding: %s\n", strerror(-r));
+                                                       goto ERROR;
+                                               }
                                        }
 
-                               } else if (fd == stderr) {
-                                       buffer = &ctx->buffers.stderr;
-                                       priority = LOG_ERR;
-
-                                       // Send any output to the default logger if no callback is set
-                                       if (ctx->communicate.out) {
-                                               callback = ctx->communicate.out;
-                                               data     = ctx->communicate.data;
-                                       } else {
-                                               callback = jail->callbacks.log;
-                                               data     = jail->callbacks.log_data;
-                                       }
+                                       // Don't fall through to log processing
+                                       continue;
 
                                } else {
                                        DEBUG(jail->pakfire, "Received invalid file descriptor %d\n", fd);
@@ -973,24 +1343,6 @@ static int pakfire_jail_wait(struct pakfire_jail* jail, struct pakfire_jail_exec
                                        goto ERROR;
                        }
 
-                       if (e & EPOLLOUT) {
-                               // Handle standard input
-                               if (fd == stdin) {
-                                       r = pakfire_jail_stream_stdin(jail, ctx, fd);
-                                       if (r) {
-                                               switch (errno) {
-                                                       // Ignore if we filled up the buffer
-                                                       case EAGAIN:
-                                                               break;
-
-                                                       default:
-                                                               ERROR(jail->pakfire, "Could not write to stdin: %m\n");
-                                                               goto ERROR;
-                                               }
-                                       }
-                               }
-                       }
-
                        // Check if any file descriptors have been closed
                        if (e & EPOLLHUP) {
                                // Remove the file descriptor
@@ -1004,12 +1356,16 @@ static int pakfire_jail_wait(struct pakfire_jail* jail, struct pakfire_jail_exec
        }
 
 ERROR:
-       if (epollfd > 0)
+       if (epollfd >= 0)
                close(epollfd);
-       if (timerfd > 0)
+       if (timerfd >= 0)
                close(timerfd);
-       if (signalfd > 0)
-               close(signalfd);
+
+       // Restore any changed terminal attributes
+       if (ctx->pty.stdin.fd >= 0)
+               pakfire_jail_restore_attrs(jail, &ctx->pty.stdin);
+       if (ctx->pty.stdout.fd >= 0)
+               pakfire_jail_restore_attrs(jail, &ctx->pty.stdout);
 
        return r;
 }
@@ -1285,8 +1641,17 @@ static int pakfire_jail_mount_networking(struct pakfire_jail* jail) {
        // Bind-mount all paths read-only
        for (const char** path = paths; *path; path++) {
                r = pakfire_bind(jail->pakfire, *path, NULL, MS_RDONLY);
-               if (r)
+               if (r) {
+                       switch (errno) {
+                               // Ignore if we don't have permission
+                               case EPERM:
+                                       continue;
+
+                               default:
+                                       break;
+                       }
                        return r;
+               }
        }
 
        return 0;
@@ -1305,7 +1670,17 @@ static int pakfire_jail_mount(struct pakfire_jail* jail, struct pakfire_jail_exe
                flags |= PAKFIRE_MOUNT_LOOP_DEVICES;
 
        // Mount all default stuff
-       r = pakfire_mount_all(jail->pakfire, flags);
+       r = pakfire_mount_all(jail->pakfire, PAKFIRE_MNTNS_OUTER, flags);
+       if (r)
+               return r;
+
+       // Populate /dev
+       r = pakfire_populate_dev(jail->pakfire, flags);
+       if (r)
+               return r;
+
+       // Mount the interpreter (if needed)
+       r = pakfire_mount_interpreter(jail->pakfire);
        if (r)
                return r;
 
@@ -1327,9 +1702,6 @@ static int pakfire_jail_mount(struct pakfire_jail* jail, struct pakfire_jail_exe
                        return r;
        }
 
-       // Log all mountpoints
-       pakfire_mount_list(jail->pakfire);
-
        return 0;
 }
 
@@ -1475,7 +1847,7 @@ static int pakfire_jail_setup_gid_mapping(struct pakfire_jail* jail, pid_t pid)
                        "0 %lu %lu\n", subgid->id, subgid->length);
        } else {
                r = pakfire_file_write(jail->pakfire, path, 0, 0, 0,
-                       "0 %lu 1\n%1 %lu %lu\n", gid, subgid->id, subgid->length);
+                       "0 %lu 1\n1 %lu %lu\n", gid, subgid->id, subgid->length);
        }
 
        if (r) {
@@ -1488,38 +1860,19 @@ static int pakfire_jail_setup_gid_mapping(struct pakfire_jail* jail, pid_t pid)
 
 static int pakfire_jail_setgroups(struct pakfire_jail* jail, pid_t pid) {
        char path[PATH_MAX];
-       int r = 1;
+       int r;
 
        // Make path
        r = pakfire_string_format(path, "/proc/%d/setgroups", pid);
        if (r)
                return r;
 
-       // Open file for writing
-       FILE* f = fopen(path, "w");
-       if (!f) {
-               ERROR(jail->pakfire, "Could not open %s for writing: %m\n", path);
-               goto ERROR;
-       }
-
-       // Write content
-       int bytes_written = fprintf(f, "deny\n");
-       if (bytes_written <= 0) {
-               ERROR(jail->pakfire, "Could not write to %s: %m\n", path);
-               goto ERROR;
-       }
-
-       r = fclose(f);
-       f = NULL;
+       r = pakfire_file_write(jail->pakfire, path, 0, 0, 0, "deny\n");
        if (r) {
-               ERROR(jail->pakfire, "Could not close %s: %m\n", path);
-               goto ERROR;
+               CTX_ERROR(jail->ctx, "Could not set setgroups to deny: %s\n", strerror(errno));
+               r = -errno;
        }
 
-ERROR:
-       if (f)
-               fclose(f);
-
        return r;
 }
 
@@ -1530,10 +1883,10 @@ static int pakfire_jail_send_signal(struct pakfire_jail* jail, int fd) {
        DEBUG(jail->pakfire, "Sending signal...\n");
 
        // Write to the file descriptor
-       ssize_t bytes_written = write(fd, &val, sizeof(val));
-       if (bytes_written < 0 || (size_t)bytes_written < sizeof(val)) {
-               ERROR(jail->pakfire, "Could not send signal: %m\n");
-               r = 1;
+       r = eventfd_write(fd, val);
+       if (r < 0) {
+               ERROR(jail->pakfire, "Could not send signal: %s\n", strerror(errno));
+               r = -errno;
        }
 
        // Close the file descriptor
@@ -1548,10 +1901,10 @@ static int pakfire_jail_wait_for_signal(struct pakfire_jail* jail, int fd) {
 
        DEBUG(jail->pakfire, "Waiting for signal...\n");
 
-       ssize_t bytes_read = read(fd, &val, sizeof(val));
-       if (bytes_read < 0 || (size_t)bytes_read < sizeof(val)) {
-               ERROR(jail->pakfire, "Error waiting for signal: %m\n");
-               r = 1;
+       r = eventfd_read(fd, &val);
+       if (r < 0) {
+               ERROR(jail->pakfire, "Error waiting for signal: %s\n", strerror(errno));
+               r = -errno;
        }
 
        // Close the file descriptor
@@ -1619,12 +1972,79 @@ static int pakfire_jail_switch_root(struct pakfire_jail* jail, const char* root)
        return 0;
 }
 
+static int pakfire_jail_open_pty(struct pakfire_jail* jail, struct pakfire_jail_exec* ctx) {
+       int r;
+
+       // Allocate a new PTY
+       ctx->pty.master.fd = posix_openpt(O_RDWR|O_NONBLOCK|O_NOCTTY|O_CLOEXEC);
+       if (ctx->pty.master.fd < 0)
+               return -errno;
+
+       // Fetch the path
+       r = ptsname_r(ctx->pty.master.fd, ctx->pty.console, sizeof(ctx->pty.console));
+       if (r)
+               return -r;
+
+       CTX_DEBUG(jail->ctx, "Allocated console at %s (%d)\n", ctx->pty.console, ctx->pty.master.fd);
+
+       // Unlock the master device
+       r = unlockpt(ctx->pty.master.fd);
+       if (r) {
+               CTX_ERROR(jail->ctx, "Could not unlock the PTY: %s\n", strerror(errno));
+               return -errno;
+       }
+
+       // Create a symlink
+       r = pakfire_symlink(jail->ctx, ctx->pty.console, "/dev/console");
+       if (r)
+               return r;
+
+       return r;
+}
+
+static int pakfire_jail_setup_terminal(struct pakfire_jail* jail, struct pakfire_jail_exec* ctx) {
+       int fd;
+       int r;
+
+       // Open a new terminal
+       fd = open("/dev/console", O_RDWR|O_NOCTTY);
+       if (fd < 0) {
+               CTX_ERROR(jail->ctx, "Failed to open a new terminal: %s\n", strerror(errno));
+               return -errno;
+       }
+
+       CTX_DEBUG(jail->ctx, "Opened a new terminal %d\n", fd);
+
+       // Connect the new terminal to standard input
+       r = dup2(fd, STDIN_FILENO);
+       if (r < 0) {
+               CTX_ERROR(jail->ctx, "Failed to open standard input: %s\n", strerror(errno));
+               return -errno;
+       }
+
+       // Connect the new terminal to standard output
+       r = dup2(fd, STDOUT_FILENO);
+       if (r < 0) {
+               CTX_ERROR(jail->ctx, "Failed to open standard output: %s\n", strerror(errno));
+               return -errno;
+       }
+
+       // Connect the new terminal to standard error
+       r = dup2(fd, STDERR_FILENO);
+       if (r < 0) {
+               CTX_ERROR(jail->ctx, "Failed to open standard error: %s\n", strerror(errno));
+               return -errno;
+       }
+
+       return 0;
+}
+
 static int pakfire_jail_child(struct pakfire_jail* jail, struct pakfire_jail_exec* ctx,
                const char* argv[]) {
        int r;
 
        // Redirect any logging to our log pipe
-       pakfire_set_log_callback(jail->pakfire, pakfire_jail_log, &ctx->pipes);
+       pakfire_ctx_set_log_callback(jail->ctx, pakfire_jail_log_redirect, &ctx->pipes);
 
        // Fetch my own PID
        pid_t pid = getpid();
@@ -1665,20 +2085,36 @@ static int pakfire_jail_child(struct pakfire_jail* jail, struct pakfire_jail_exe
        uid_t euid = geteuid();
        gid_t egid = getegid();
 
-       DEBUG(jail->pakfire, "  UID: %d (effective %d)\n", uid, euid);
-       DEBUG(jail->pakfire, "  GID: %d (effective %d)\n", gid, egid);
+       DEBUG(jail->pakfire, "  UID: %u (effective %u)\n", uid, euid);
+       DEBUG(jail->pakfire, "  GID: %u (effective %u)\n", gid, egid);
 
-       // Check if we are (effectively running as root)
+       // Log all mountpoints
+       pakfire_mount_list(jail->ctx);
+
+       // Fail if we are not PID 1
+       if (pid != 1) {
+               CTX_ERROR(jail->ctx, "Child process is not PID 1\n");
+               return 126;
+       }
+
+       // Fail if we are not running as root
        if (uid || gid || euid || egid) {
                ERROR(jail->pakfire, "Child process is not running as root\n");
                return 126;
        }
 
+       const int socket_send = pakfire_jail_get_pipe_to_write(jail, &ctx->socket);
+
+       // Mount all default stuff
+       r = pakfire_mount_all(jail->pakfire, PAKFIRE_MNTNS_INNER, 0);
+       if (r)
+               return 126;
+
        const char* root = pakfire_get_path(jail->pakfire);
        const char* arch = pakfire_get_effective_arch(jail->pakfire);
 
        // Change mount propagation to slave to receive anything from the parent namespace
-       r = pakfire_mount_change_propagation(jail->pakfire, MS_SLAVE, "/");
+       r = pakfire_mount_change_propagation(jail->ctx, "/", MS_SLAVE);
        if (r)
                return r;
 
@@ -1688,7 +2124,7 @@ static int pakfire_jail_child(struct pakfire_jail* jail, struct pakfire_jail_exe
                return r;
 
        // Change mount propagation to private
-       r = pakfire_mount_change_propagation(jail->pakfire, MS_PRIVATE, root);
+       r = pakfire_mount_change_propagation(jail->ctx, root, MS_PRIVATE);
        if (r)
                return r;
 
@@ -1733,47 +2169,45 @@ static int pakfire_jail_child(struct pakfire_jail* jail, struct pakfire_jail_exe
                }
        }
 
-       // Close other end of log pipes
-       close(ctx->pipes.log_INFO[0]);
-       close(ctx->pipes.log_ERROR[0]);
-#ifdef ENABLE_DEBUG
-       close(ctx->pipes.log_DEBUG[0]);
-#endif /* ENABLE_DEBUG */
-
-       // Connect standard input
-       if (ctx->pipes.stdin[0] >= 0) {
-               r = dup2(ctx->pipes.stdin[0], STDIN_FILENO);
-               if (r < 0) {
-                       ERROR(jail->pakfire, "Could not connect fd %d to stdin: %m\n",
-                               ctx->pipes.stdin[0]);
+       // Create a new session
+       r = setsid();
+       if (r < 0) {
+               CTX_ERROR(jail->ctx, "Could not create a new session: %s\n", strerror(errno));
+               return r;
+       }
 
-                       return 1;
-               }
+       // Allocate a new PTY
+       r = pakfire_jail_open_pty(jail, ctx);
+       if (r) {
+               CTX_ERROR(jail->ctx, "Could not allocate a new PTY: %s\n", strerror(-r));
+               return r;
        }
 
-       // Connect standard output and error
-       if (ctx->pipes.stdout[1] >= 0 && ctx->pipes.stderr[1] >= 0) {
-               r = dup2(ctx->pipes.stdout[1], STDOUT_FILENO);
-               if (r < 0) {
-                       ERROR(jail->pakfire, "Could not connect fd %d to stdout: %m\n",
-                               ctx->pipes.stdout[1]);
+       // Send the PTY master to the parent process
+       r = pakfire_jail_send_fd(jail, socket_send, ctx->pty.master.fd);
+       if (r) {
+               CTX_ERROR(jail->ctx, "Failed sending the PTY master to the parent: %s\n", strerror(-r));
+               return r;
+       }
 
-                       return 1;
-               }
+       // Setup the terminal
+       r = pakfire_jail_setup_terminal(jail, ctx);
+       if (r)
+               return r;
 
-               r = dup2(ctx->pipes.stderr[1], STDERR_FILENO);
-               if (r < 0) {
-                       ERROR(jail->pakfire, "Could not connect fd %d to stderr: %m\n",
-                               ctx->pipes.stderr[1]);
+       // Close the master of the PTY
+       close(ctx->pty.master.fd);
+       ctx->pty.master.fd = -1;
 
-                       return 1;
-               }
+       // Close the socket
+       close(socket_send);
 
-               // Close the pipe (as we have moved the original file descriptors)
-               pakfire_jail_close_pipe(jail, ctx->pipes.stdin);
-               pakfire_jail_close_pipe(jail, ctx->pipes.stdout);
-               pakfire_jail_close_pipe(jail, ctx->pipes.stderr);
-       }
+       // Close other end of log pipes
+       close(ctx->pipes.log_INFO[0]);
+       close(ctx->pipes.log_ERROR[0]);
+#ifdef ENABLE_DEBUG
+       close(ctx->pipes.log_DEBUG[0]);
+#endif /* ENABLE_DEBUG */
 
        // Reset open file limit (http://0pointer.net/blog/file-descriptor-limits.html)
        r = pakfire_rlimit_reset_nofile(jail->pakfire);
@@ -1800,7 +2234,7 @@ static int pakfire_jail_child(struct pakfire_jail* jail, struct pakfire_jail_exe
 
        // Log argv
        for (unsigned int i = 0; argv[i]; i++)
-               DEBUG(jail->pakfire, "  argv[%d] = %s\n", i, argv[i]);
+               DEBUG(jail->pakfire, "  argv[%u] = %s\n", i, argv[i]);
 
        // exec() command
        r = execvpe(argv[0], (char**)argv, jail->env);
@@ -1846,10 +2280,14 @@ static int __pakfire_jail_exec(struct pakfire_jail* jail, const char* argv[],
        struct pakfire_jail_exec ctx = {
                .flags = flags,
 
+               .socket = { -1, -1 },
+
                .pipes = {
-                       .stdin  = { -1, -1 },
-                       .stdout = { -1, -1 },
-                       .stderr = { -1, -1 },
+                       .log_INFO  = { -1, -1 },
+                       .log_ERROR = { -1, -1 },
+#ifdef ENABLE_DEBUG
+                       .log_DEBUG = { -1, -1 },
+#endif /* ENABLE_DEBUG */
                },
 
                .communicate = {
@@ -1859,6 +2297,19 @@ static int __pakfire_jail_exec(struct pakfire_jail* jail, const char* argv[],
                },
 
                .pidfd = -1,
+
+               // PTY
+               .pty = {
+                       .master = {
+                               .fd = -1,
+                       },
+                       .stdin = {
+                               .fd = -1,
+                       },
+                       .stdout = {
+                               .fd = -1,
+                       },
+               },
        };
 
        DEBUG(jail->pakfire, "Executing jail...\n");
@@ -1877,24 +2328,12 @@ static int __pakfire_jail_exec(struct pakfire_jail* jail, const char* argv[],
                return -1;
        }
 
-       // Create pipes to communicate with child process if we are not running interactively
-       if (!interactive) {
-               // stdin (only if callback is set)
-               if (ctx.communicate.in) {
-                       r = pakfire_jail_setup_pipe(jail, &ctx.pipes.stdin, 0);
-                       if (r)
-                               goto ERROR;
-               }
-
-               // stdout
-               r = pakfire_jail_setup_pipe(jail, &ctx.pipes.stdout, 0);
-               if (r)
-                       goto ERROR;
-
-               // stderr
-               r = pakfire_jail_setup_pipe(jail, &ctx.pipes.stderr, 0);
-               if (r)
-                       goto ERROR;
+       // Create a UNIX domain socket
+       r = socketpair(AF_UNIX, SOCK_DGRAM|SOCK_CLOEXEC, 0, ctx.socket);
+       if (r < 0) {
+               CTX_ERROR(jail->ctx, "Could not create UNIX socket: %s\n", strerror(errno));
+               r = -errno;
+               goto ERROR;
        }
 
        // Setup pipes for logging
@@ -2006,26 +2445,23 @@ ERROR:
        // Destroy the temporary cgroup (if any)
        if (ctx.cgroup) {
                // Read cgroup stats
-               r = pakfire_cgroup_stat(ctx.cgroup, &ctx.cgroup_stats);
-               if (r) {
-                       ERROR(jail->pakfire, "Could not read cgroup stats: %m\n");
-               } else {
-                       pakfire_cgroup_stat_dump(ctx.cgroup, &ctx.cgroup_stats);
-               }
-
+               pakfire_cgroup_stat(ctx.cgroup, &ctx.cgroup_stats);
+               pakfire_cgroup_stat_dump(ctx.cgroup, &ctx.cgroup_stats);
                pakfire_cgroup_destroy(ctx.cgroup);
                pakfire_cgroup_unref(ctx.cgroup);
        }
 
        // Close any file descriptors
-       pakfire_jail_close_pipe(jail, ctx.pipes.stdin);
-       pakfire_jail_close_pipe(jail, ctx.pipes.stdout);
-       pakfire_jail_close_pipe(jail, ctx.pipes.stderr);
-       if (ctx.pidfd)
+       if (ctx.pidfd >= 0)
                close(ctx.pidfd);
+       if (ctx.pty.master.fd >= 0)
+               close(ctx.pty.master.fd);
        pakfire_jail_close_pipe(jail, ctx.pipes.log_INFO);
        pakfire_jail_close_pipe(jail, ctx.pipes.log_ERROR);
+#ifdef ENABLE_DEBUG
        pakfire_jail_close_pipe(jail, ctx.pipes.log_DEBUG);
+#endif /* ENABLE_DEBUG */
+       pakfire_jail_close_pipe(jail, ctx.socket);
 
        return exit;
 }
@@ -2066,7 +2502,7 @@ int pakfire_jail_exec_script(struct pakfire_jail* jail,
        const char* root = pakfire_get_path(jail->pakfire);
 
        // Write the scriptlet to disk
-       r = pakfire_path_join(path, root, PAKFIRE_TMP_DIR "/pakfire-script.XXXXXX");
+       r = pakfire_path_append(path, root, PAKFIRE_TMP_DIR "/pakfire-script.XXXXXX");
        if (r)
                goto ERROR;