Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Handle missing RDPVersion numbers thrown by some RDP clients #477

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

luciaprime54
Copy link
Contributor

A recent update of some clients such as Remmina on Linux recently updated the RDP version code to 0x80011 and threw an exception crashing unnecessarily.

Rather than waiting to hardcode the next version number, handle the ones we need to and create a catch-all for the ones we haven't seen.

PR also adds the latest version I saw in the wild: 0x80011

@energister
Copy link

This PR is attempt to address #475.

Unfortunately, it doesn't help me: I'm getting errors and disconnection on the Windows 11 target host (see Event Viewer) and connection timeout at the Windows 11 mstsc client on the other side.
image

@jmvermeulen
Copy link

jmvermeulen commented Feb 26, 2025

Can confirm this issue, it's the Windows 11 client that causes the problem. Using an older client OS this error does not apply.

Edit: can confirm that this patch works too!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants