Merge pull request #896 from hardening/hardening5

Don't break connection when receiving an invalid synchronize packet
This commit is contained in:
Marc-André Moreau 2013-01-14 05:08:22 -08:00
commit 8b3b635d99
1 changed files with 4 additions and 1 deletions

View File

@ -192,7 +192,10 @@ static BOOL fastpath_recv_update_common(rdpFastPath* fastpath, STREAM* s)
static BOOL fastpath_recv_update_synchronize(rdpFastPath* fastpath, STREAM* s)
{
return stream_skip(s, 2); /* size (2 bytes), MUST be set to zero */
/* server 2008 can send invalid synchronize packet with missing padding,
so don't return FALSE even if the packet is invalid */
stream_skip(s, 2); /* size (2 bytes), MUST be set to zero */
return TRUE;
}
static BOOL fastpath_recv_update(rdpFastPath* fastpath, BYTE updateCode, UINT32 size, STREAM* s)