Merge pull request #4562 from nextcloud/bugfix/finder-sync-overflow

Fix crash caused by overflow in FinderSyncExtension
nextcloud · May 19, 2022 · cb8cc2a · cb8cc2a
2 parents 5523330 + 1f04073
commit cb8cc2a
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/shell_integration/MacOSX/OwnCloudFinderSync/FinderSyncExt/LocalSocketClient.m b/shell_integration/MacOSX/OwnCloudFinderSync/FinderSyncExt/LocalSocketClient.m
@@ -207,9 +207,9 @@ - (void)writeToSocket
     NSLog(@"About to write %li bytes from outbuffer to socket.", [self.outBuffer length]);
 
     long bytesWritten = write(self.sock, [self.outBuffer bytes], [self.outBuffer length]);
-    char lineWritten[4096];
+    char lineWritten[[self.outBuffer length]];
     memcpy(lineWritten, [self.outBuffer bytes], [self.outBuffer length]);
-    NSLog(@"Wrote %li bytes to socket. Line was: '%@'", bytesWritten, [NSString stringWithUTF8String:lineWritten]);
+    NSLog(@"Wrote %li bytes to socket. Line written was: '%@'", bytesWritten, [NSString stringWithUTF8String:lineWritten]);
 
     if(bytesWritten == 0) {
         // 0 means we reached "end of file" and thus the socket was closed. So let's restart it