1

Closed

RDPBCGR Slow path input not parsed properly

description

The current version of the parsers incorrectly parse slow path input PDU. It skips the numberEvents and pad2Octets fields, which cases all the other fields to be incorrect. The fix is very simple, and I would edit the code myself, but I don't know how to commit changes on CodePlex. In the rdpbcgr.npl file, replace:
case 28:
                [Property.RDPBCGRPacketType = "TsInputEvent"]
                RDPBCGRTsInputEvent TsInputEvent;
with
case 28:
                [Property.RDPBCGRPacketType = "TsInputEvent"]
                RDPBCGRTsInputPduData TsInputEvent;
I have tested this on my local copy and this one change fixes the issue.
Thanks!

file attachments

Closed Aug 28, 2012 at 5:19 AM by Jessicawu
Thanks for your reporting this issue, I have reslove the issue and you can get it in sep release.

comments

rachelhu wrote Jul 30, 2012 at 2:59 AM

Thanks for reporting this issue for us. We will do the win8 RDP series protocol alignment for parser during Nov. Then, do the public release. I think you can verify this issue that time.Would you mind to attach the capture you have related with this issue. Thanks a lot.

wrote Jul 30, 2012 at 4:50 AM

felipet1326 wrote Jul 30, 2012 at 4:50 AM

Thanks for replying so quickly. You're welcome. This issue is not related to Windows 8. It happens even when connecting to XP computers. Note that you would probably never see it in practice with mstsc.exe because these packets would be encrypted and/or compressed, but neither is required. Here is a capture containing two frames with valid input data. Open them with the current parsers, then try the change I recommended, and reopen it.
Sorry, this capture has lots of extra baggage, but if I only send you the relevant frames, it doesn't recognize it properly. Frames #131 and 118 among many others, demonstrate the issue.

wrote Aug 28, 2012 at 5:19 AM

Jessicawu wrote Aug 28, 2012 at 10:50 AM

Please get the changeset 81160 to verfy the issue.Thanks!

wrote Feb 22, 2013 at 12:34 AM

wrote May 16, 2013 at 12:14 PM