From 3e28c9adf4bfe2f4792557ee45684181bd6e6d1c Mon Sep 17 00:00:00 2001 From: Gerd Hoffmann Date: Mon, 27 Jul 2009 17:10:48 +0200 Subject: [PATCH] vnc: fix copyrect screen corruption When sending a copyrect command to the vnc client, we must also update the local server surface. Otherwise the server's and the client's idea of the screen content run out of sync and screen updates don't work correctly. Signed-off-by: Anthony Liguori --- vnc.c | 22 ++++++++++++++++++++++ 1 file changed, 22 insertions(+) diff --git a/vnc.c b/vnc.c index dfcf34c01e..903dd953d9 100644 --- a/vnc.c +++ b/vnc.c @@ -654,6 +654,11 @@ static void send_framebuffer_update(VncState *vs, int x, int y, int w, int h) static void vnc_copy(VncState *vs, int src_x, int src_y, int dst_x, int dst_y, int w, int h) { + uint8_t *src_row; + uint8_t *dst_row; + int y,pitch,depth; + + /* send bitblit op to the vnc client */ vnc_write_u8(vs, 0); /* msg id */ vnc_write_u8(vs, 0); vnc_write_u16(vs, 1); /* number of rects */ @@ -661,6 +666,23 @@ static void vnc_copy(VncState *vs, int src_x, int src_y, int dst_x, int dst_y, i vnc_write_u16(vs, src_x); vnc_write_u16(vs, src_y); vnc_flush(vs); + + /* do bitblit op on the local surface too */ + pitch = ds_get_linesize(vs->ds); + depth = ds_get_bytes_per_pixel(vs->ds); + src_row = vs->server.ds->data + pitch * src_y + depth * src_x; + dst_row = vs->server.ds->data + pitch * dst_y + depth * dst_x; + if (dst_y > src_y) { + /* copy backwards */ + src_row += pitch * (h-1); + dst_row += pitch * (h-1); + pitch = -pitch; + } + for (y = 0; y < h; y++) { + memmove(dst_row, src_row, w * depth); + src_row += pitch; + dst_row += pitch; + } } static void vnc_dpy_copy(DisplayState *ds, int src_x, int src_y, int dst_x, int dst_y, int w, int h)