Fix for #4330
authorDavid Fort <contact@hardening-consulting.com>
Sat, 23 Dec 2017 12:50:54 +0000 (13:50 +0100)
committerDavid Fort <contact@hardening-consulting.com>
Sat, 23 Dec 2017 12:50:54 +0000 (13:50 +0100)
commit191b8f950fcf77aa8c90b819f3d5b2dc3958ecfa
tree38ce97924c587545ec646de5bbd0183a9d1d25c1
parent58b5573c82be8d2e16424c438ebe40b4c53a57ef
Fix for #4330

Since ec027bf dynamic resolution is broken when used with egfx. Before that commit
we were tracking a server sent resize by setting a DesktopResize callback. This callback
is called when the desktop is resized by the server. Anyway the problem was that when this
callback is called, the activation sequence is not always completed, which were leading to
some freeze with 2012r2 servers (sending packets before the sequence is finished).
So with the faulty commit, we are tracking server resizes by subscribing to the Actived
event, that is called at the end of a reactivation sequence, so we're sure to not send packets
when not fully activated.
Anyway the issue that shows on (#4330) is that when you use egfx, no reactivation sequence happens,
the server only sends a ResetGraphics message with the new size, and so we miss the resized event.
This fix introduces a new GraphicsReset event, makes the display channel subscribe to that event,
and react accordingly.
channels/rdpgfx/client/rdpgfx_codec.c
channels/rdpgfx/client/rdpgfx_main.c
client/X11/xf_disp.c
include/freerdp/event.h
libfreerdp/core/freerdp.c