aboutsummaryrefslogtreecommitdiffstats
path: root/common/rfb
diff options
context:
space:
mode:
authorPierre Ossman <ossman@cendio.se>2021-01-14 13:07:56 +0100
committerPierre Ossman <ossman@cendio.se>2021-01-14 13:07:56 +0100
commit6d1603c3da504f76a4f7364f89e683a8e1632fdc (patch)
tree642c1bc9b6321746b5fffc5d8f920992272efad6 /common/rfb
parent16e39980dcd2ce882a44437ee4b3646308136d87 (diff)
downloadtigervnc-6d1603c3da504f76a4f7364f89e683a8e1632fdc.tar.gz
tigervnc-6d1603c3da504f76a4f7364f89e683a8e1632fdc.zip
Fix handling of bad update requests
We computed a safe area if a client gave us a bogus one, but we didn't actually use it. Fix this properly and make sure we don't pass on bad coordinates further.
Diffstat (limited to 'common/rfb')
-rw-r--r--common/rfb/VNCSConnectionST.cxx2
1 files changed, 1 insertions, 1 deletions
diff --git a/common/rfb/VNCSConnectionST.cxx b/common/rfb/VNCSConnectionST.cxx
index 9b5ead9f..1db94cac 100644
--- a/common/rfb/VNCSConnectionST.cxx
+++ b/common/rfb/VNCSConnectionST.cxx
@@ -628,7 +628,7 @@ void VNCSConnectionST::framebufferUpdateRequest(const Rect& r,bool incremental)
// Just update the requested region.
// Framebuffer update will be sent a bit later, see processMessages().
- Region reqRgn(r);
+ Region reqRgn(safeRect);
if (!incremental || !continuousUpdates)
requested.assign_union(reqRgn);