Merge pull request #1844 from caramorsimon/tokenrouting
Test for CRLF (0x0D0A) termination on the routing token
This commit is contained in:
commit
6f8fda247b
@ -704,9 +704,19 @@ BOOL nego_send_negotiation_request(rdpNego* nego)
|
||||
if (nego->RoutingToken)
|
||||
{
|
||||
Stream_Write(s, nego->RoutingToken, nego->RoutingTokenLength);
|
||||
Stream_Write_UINT8(s, 0x0D); /* CR */
|
||||
Stream_Write_UINT8(s, 0x0A); /* LF */
|
||||
length += nego->RoutingTokenLength + 2;
|
||||
/* Ensure Routing Token is correctly terminated - may already be present in string */
|
||||
if (nego->RoutingTokenLength>2 && (nego->RoutingToken[nego->RoutingTokenLength-2]==0x0D && nego->RoutingToken[nego->RoutingTokenLength-1]==0x0A))
|
||||
{
|
||||
DEBUG_NEGO("Routing token looks correctly terminated - use verbatim");
|
||||
length +=nego->RoutingTokenLength;
|
||||
}
|
||||
else
|
||||
{
|
||||
DEBUG_NEGO("Adding terminating CRLF to routing token");
|
||||
Stream_Write_UINT8(s, 0x0D); /* CR */
|
||||
Stream_Write_UINT8(s, 0x0A); /* LF */
|
||||
length += nego->RoutingTokenLength + 2;
|
||||
}
|
||||
}
|
||||
else if (nego->cookie)
|
||||
{
|
||||
|
Loading…
Reference in New Issue
Block a user