GUACAMOLE-1846: Fix __write_all() to return length as documented.

apache · Aug 23, 2023 · 6265817 · 6265817
1 parent a2c28c9
commit 6265817
Showing 1 changed file with 4 additions and 3 deletions.
diff --git 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;
 
     }