From a7443a521c3c19e2b1aeec6ee181b7a32fe86674 Mon Sep 17 00:00:00 2001
From: James Muehlner <james.muehlner@guac-dev.org>
Date: Tue, 22 Aug 2023 17:46:53 +0000
Subject: [PATCH] GUACAMOLE-1846: Fix __write_all() to return length as
 documented.

---
 src/guacd/connection.c | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

diff --git a/src/guacd/connection.c b/src/guacd/connection.c
index a5b98adf..1bb99dc0 100644
--- a/src/guacd/connection.c
+++ b/src/guacd/connection.c
@@ -67,13 +67,14 @@
 static int __write_all(int fd, char* buffer, int length) {
 
     /* Repeatedly write() until all data is written */
-    while (length > 0) {
+    int remaining_length = length;
+    while (remaining_length > 0) {
 
-        int written = write(fd, buffer, length);
+        int written = write(fd, buffer, remaining_length);
         if (written < 0)
             return -1;
 
-        length -= written;
+        remaining_length -= written;
         buffer += written;
 
     }