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

Workaround for [MS-RDPBCGR] 2.2.9.2.3 Frame Marker Command (TS_FRAME_… #7714

Merged
merged 1 commit into from
Mar 9, 2022

Conversation

akallabeth
Copy link
Member

…MARKER)

Connections with windows 2016 and 2019 sometimes receive short
frame marker. Ignore these to prevent disconnects

@akallabeth akallabeth added this to the next milestone Mar 9, 2022
@freerdp-bot
Copy link

Refer to this link for build results (access rights to CI server needed):
https://ci.freerdp.com//job/PullRequestTester/7231/

…MARKER)

Connections with  windows 2016 and 2019 sometimes receive short
frame marker. Ignore these to prevent disconnects
@freerdp-bot
Copy link

Refer to this link for build results (access rights to CI server needed):
https://ci.freerdp.com//job/PullRequestTester/7232/

@akallabeth
Copy link
Member Author

@Fantu please try this out and check, if the disconnections are gone. It should print a very specific warning whenever this issue happens.

@Fantu
Copy link
Contributor

Fantu commented Mar 9, 2022

@akallabeth big thanks, applied this to 2.6.1, reproduced one time:
[12:38:27:588] [1153:1168] [WARN][com.freerdp.core.surface] - [SERVER-BUG]: got 0, expected 4 bytes. [MS-RDPBCGR] 2.2.9.2.3 Frame Marker Command (TS_FRAME_MARKER) is missing frameId, ignoring
but didn't disconnect this time, so this seems solved
now I started to update some users thin client to check if there are different cases

@akallabeth
Copy link
Member Author

@Fantu ok, thank you. Could you add some information about the server side? I´ll try to get some more details regarding this, but would need the server version & settings to do that.

@Fantu
Copy link
Contributor

Fantu commented Mar 9, 2022

I reproduced it for example connecting to:

  • windows server 2019 standard build 17763.2565 (I wait users confirm that after update disconnect is not reproducible anymore) (on one customers we have 2 twins virtual server both with build 17763.2565 one have disconnect issue one not, tried to find different settings windows side but don't found any) other 2 customers have disconnect issue with W2019 but I not had debug on them to be sure is this issue the cause, I'll update some thin clients and ask to users to tell me if the disconnect persist (and in that case I'll take a debug) and if not persist I'll have the confirm was this
  • Windows 10 Pro build 19043.1526 (seems solved, don't disconnected anymore for now and I still testing it keep connecting from thin client)
  • Windows 10 Pro build 19041.1415 (where I checked the log, saw the warning of problem reproduce but didn't disconnect)

sorry for my bad english

@akallabeth akallabeth marked this pull request as ready for review March 9, 2022 14:52
@akallabeth akallabeth merged commit 91ef44e into FreeRDP:master Mar 9, 2022
@akallabeth akallabeth deleted the frame_marker_bugfix branch March 9, 2022 14:52
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.

None yet

3 participants