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

wrong message order in [MS-RDPEFS] setup #2834

Closed
akallabeth opened this issue Oct 23, 2023 · 2 comments · Fixed by #2838
Closed

wrong message order in [MS-RDPEFS] setup #2834

akallabeth opened this issue Oct 23, 2023 · 2 comments · Fixed by #2838
Labels
confirmed confirmed reproduction

Comments

@akallabeth
Copy link

  1. Got a report Drive redirection not working in Alt Linux FreeRDP/FreeRDP#9501
  2. found
    devredir_send_server_user_logged_on();
  3. [MS-RDPEFS] tells that this is the wrong state you´re sending the RDPDR_USER_LOGGEDON_PDU
@matt335672
Copy link
Member

Thanks @akallabeth

I'm out of time at the mo', but I'll look later in the week. Your OP appears to have sent all relevant info.

@matt335672
Copy link
Member

I can confirm xrdp is not conforming to [MS-RDPEFS], so it's almost certainly an xrdp bug.

It happens with FreeRDP 2.11.2 (Alt Linux 10.1) but not 2.9.0.

By moving the devredir_send_server_user_logged_on() down to after we've received the client core capability response (as the specification) I can get the redirection working again.

That's a quick test. I'm not testing the client has RDPDR_USER_LOGGEDON_PDU set yet. I'll also need to re-test with existing working clients then I can issue a patch.

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

Successfully merging a pull request may close this issue.

2 participants