Fixed length checking in StatFile

If a symlink resolves to a path that is longer than the maximum allowed by the protocol, currently 4088 bytes, then an error response is generated.

Previously the server side would ignore an error from SendTransaction() due to the large size and the client-side would hang/timeout waiting for a response.

Ticket: ENT-13542
Changelog: title

Author: craigcomstock

cf-serverd/server_common.c

@@ -874,10 +874,15 @@ int StatFile(ServerConnectionState *conn, char *sendbuffer, char *ofilename)
 
     memset(sendbuffer, 0, CF_MSGSIZE);
 
+    // +3 because we need to prepend 'OK:' to the path
+    if (strlen(linkbuf)+3 > CF_MSGSIZE) {
+        snprintf(sendbuffer, CF_MSGSIZE, "BAD: Symlink resolves to a path too long (%ld) to send over the protocol.", strlen(linkbuf)+3);
+        SendTransaction(conn->conn_info, sendbuffer, 0, CF_DONE);
+        return -1;
+    }
     if (cfst.cf_readlink != NULL)
     {
-        strcpy(sendbuffer, "OK:");
-        strcat(sendbuffer, cfst.cf_readlink);
+        assert(snprintf(sendbuffer, CF_MSGSIZE, "OK:%s", linkbuf));
     }
     else
     {