Fix -Wunused-result problems

1 files changed, 11 insertions, 3 deletions

diff --git a/sway/commands.c b/sway/commands.c
index c1390016..3092239c 100644
--- a/sway/commands.c
+++ b/sway/commands.c
@@ -509,7 +509,9 @@ static struct cmd_results *cmd_exec_always(int argc, char **argv) {
         sway_log(L_DEBUG, "Executing %s", cmd);
 
         int fd[2];
-        pipe(fd);
+        if (pipe(fd) != 0) {
+                sway_log(L_ERROR, "Unable to create pipe for fork");
+        }
 
         pid_t pid;
         pid_t *child = malloc(sizeof(pid_t)); // malloc'd so that Linux can avoid copying the process space
@@ -522,14 +524,20 @@ static struct cmd_results *cmd_exec_always(int argc, char **argv) {
                         /* Not reached */
                 }
                 close(fd[0]);
-                write(fd[1], child, sizeof(pid_t));
+                ssize_t s = 0;
+                while ((size_t)s < sizeof(pid_t)) {
+                        s += write(fd[1], ((uint8_t *)child) + s, sizeof(pid_t));
+                }
                 close(fd[1]);
                 _exit(0); // Close child process
         } else if (pid < 0) {
                 return cmd_results_new(CMD_FAILURE, "exec_always", "Command failed (sway could not fork).");
         }
         close(fd[1]); // close write
-        read(fd[0], child, sizeof(pid_t));
+        ssize_t s = 0;
+        while ((size_t)s < sizeof(pid_t)) {
+                s += read(fd[0], ((uint8_t *)child) + s, sizeof(pid_t));
+        }
         close(fd[0]);
         // cleanup child process
         wait(0);