Re: [Qemu-devel] [PATCH] vnc: fix screen updates

[Anthony PERARD]

On Fri, Jun 13, 2014 at 10:25:23AM +0200, Gerd Hoffmann wrote:
> Bug was added by 38ee14f4f33f8836fc0e209ca59c6ae8c6edf380.
> vnc_jobs_join call is missing in one code path.
> 
> Reported-by: Anthony PERARD <address@hidden>
> Signed-off-by: Gerd Hoffmann <address@hidden>

I've tested the patch, it works.

Thanks!

> ---
>  ui/vnc.c | 3 +++
>  1 file changed, 3 insertions(+)
> 
> diff --git a/ui/vnc.c b/ui/vnc.c
> index 6c9d4f3..aac93f0 100644
> --- a/ui/vnc.c
> +++ b/ui/vnc.c
> @@ -935,6 +935,9 @@ static int vnc_update_client(VncState *vs, int has_dirty, 
> bool sync)
>          }
>  
>          vnc_job_push(job);
> +        if (sync) {
> +            vnc_jobs_join(vs);
> +        }
>          vs->force_update = 0;
>          return n;
>      }

-- 
Anthony PERARD