Patchwork [2/5] vnc: send desktopresize event as reply to set encodings

login
register
mail settings
Submitter Gerd Hoffmann
Date May 25, 2010, 4:25 p.m.
Message ID <1274804720-518-3-git-send-email-kraxel@redhat.com>
Download mbox | patch
Permalink /patch/53559/
State New
Headers show

Comments

Gerd Hoffmann - May 25, 2010, 4:25 p.m.
In case the desktop did resize while the vnc connection setup was still
in progress the client isn't informed about it.  Send a desktop resize
event as soon as the client told us it can handle deskop resize via set
encodings message to make sure the client us up to date.

Signed-off-by: Gerd Hoffmann <kraxel@redhat.com>
---
 vnc.c |    1 +
 1 files changed, 1 insertions(+), 0 deletions(-)
Corentin Chary - May 25, 2010, 4:41 p.m.
On Tue, May 25, 2010 at 6:25 PM, Gerd Hoffmann <kraxel@redhat.com> wrote:
> In case the desktop did resize while the vnc connection setup was still
> in progress the client isn't informed about it.  Send a desktop resize
> event as soon as the client told us it can handle deskop resize via set
> encodings message to make sure the client us up to date.

I had a similar patch on my queue but yours is probably cleaner :).

Patch

diff --git a/vnc.c b/vnc.c
index aaebe24..0e0e566 100644
--- a/vnc.c
+++ b/vnc.c
@@ -1709,6 +1709,7 @@  static void set_encodings(VncState *vs, int32_t *encodings, size_t n_encodings)
             break;
         }
     }
+    vnc_desktop_resize(vs);
     check_pointer_type_change(&vs->mouse_mode_notifier);
 }