Old-Ding opened a new pull request, #3595: URL: https://github.com/apache/nuttx-apps/pull/3595
## Summary - keep the original RFB ServerInit desktop-name length separate from the copied length - drain any desktop-name bytes that do not fit in `conn->name` - reuse a small discard helper for skipped RFB payloads ## Why The RFB ServerInit desktop name is a length-prefixed field. When the server advertises a name longer than the local buffer, vncviewer truncated the copy length before reading from the socket. That left the remaining name bytes queued as if they were the next RFB message, which can desynchronize subsequent framebuffer update parsing. ## Testing - `git diff --check origin/master..HEAD` - `git show --check --stat --oneline HEAD` - attempted `checkpatch.sh`; not run because this Windows environment has no usable WSL/bash shell -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
