Index: unix/vncviewer/CConn.cxx =================================================================== --- unix/vncviewer/CConn.cxx (revision 4003) +++ unix/vncviewer/CConn.cxx (revision 4004) @@ -33,6 +33,7 @@ #include #include #include +#include #include "TXViewport.h" #include "DesktopWindow.h" @@ -62,7 +63,7 @@ encodingChange(false), sameMachine(false), fullScreen(::fullScreen), ctrlDown(false), altDown(false), menuKeysym(0), menu(dpy, this), options(dpy, this), about(dpy), info(dpy), - reverseConnection(reverse), firstUpdate(true) + reverseConnection(reverse), firstUpdate(true), pendingUpdate(false) { CharArray menuKeyStr(menuKey.getData()); menuKeysym = XStringToKeysym(menuKeyStr.buf); @@ -306,8 +307,11 @@ // one. We cannot do this if we're in the middle of a format change // though. void CConn::framebufferUpdateStart() { - if (!formatChange) + if (!formatChange) { + pendingUpdate = true; requestNewUpdate(); + } else + pendingUpdate = false; } // framebufferUpdateEnd() is called at the end of an update. @@ -367,7 +371,7 @@ // A format change prevented us from sending this before the update, // so make sure to send it now. - if (formatChange) + if (formatChange && !pendingUpdate) requestNewUpdate(); // Compute new settings based on updated bandwidth values @@ -536,8 +540,11 @@ break; case ID_REFRESH: menu.unmap(); - writer()->writeFramebufferUpdateRequest(Rect(0, 0, cp.width, cp.height), - false); + if (!formatChange) { + writer()->writeFramebufferUpdateRequest(Rect(0, 0, cp.width, cp.height), + false); + pendingUpdate = true; + } break; case ID_F8: menu.unmap(); @@ -840,6 +847,10 @@ void CConn::requestNewUpdate() { if (formatChange) { + + /* Catch incorrect requestNewUpdate calls */ + assert(pendingUpdate == false); + if (fullColour) { desktop->setPF(fullColourPF); } else { Index: unix/vncviewer/CConn.h =================================================================== --- unix/vncviewer/CConn.h (revision 4003) +++ unix/vncviewer/CConn.h (revision 4004) @@ -132,6 +132,7 @@ InfoDialog info; bool reverseConnection; bool firstUpdate; + bool pendingUpdate; }; #endif