[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 15/19] vnc: fix local state init
From: |
Daniel P. Berrange |
Subject: |
Re: [Qemu-devel] [PATCH 15/19] vnc: fix local state init |
Date: |
Fri, 30 Oct 2015 21:26:54 +0900 |
User-agent: |
Mutt/1.5.24 (2015-08-30) |
On Fri, Oct 30, 2015 at 12:10:10PM +0100, Gerd Hoffmann wrote:
It isn't entirely obvious what bug is being fixed here, perhaps
a little more info in the commit message would help.
> Signed-off-by: Gerd Hoffmann <address@hidden>
> ---
> ui/vnc-jobs.c | 8 ++++----
> 1 file changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/ui/vnc-jobs.c b/ui/vnc-jobs.c
> index fd9ed39..12389cc 100644
> --- a/ui/vnc-jobs.c
> +++ b/ui/vnc-jobs.c
> @@ -185,6 +185,9 @@ void vnc_jobs_consume_buffer(VncState *vs)
> */
> static void vnc_async_encoding_start(VncState *orig, VncState *local)
> {
> + buffer_init(&local->output, "vnc-worker-output");
> + local->csock = -1; /* Don't do any network work on this thread */
> +
> local->vnc_encoding = orig->vnc_encoding;
> local->features = orig->features;
> local->vd = orig->vd;
> @@ -196,7 +199,6 @@ static void vnc_async_encoding_start(VncState *orig,
> VncState *local)
> local->zlib = orig->zlib;
> local->hextile = orig->hextile;
> local->zrle = orig->zrle;
> - local->csock = -1; /* Don't do any network work on this thread */
> }
>
> static void vnc_async_encoding_end(VncState *orig, VncState *local)
> @@ -212,12 +214,10 @@ static int vnc_worker_thread_loop(VncJobQueue *queue)
> {
> VncJob *job;
> VncRectEntry *entry, *tmp;
> - VncState vs;
> + VncState vs = {};
Ok, so we'd not initialized memory in VncState to all zeros before.
> int n_rectangles;
> int saved_offset;
>
> - buffer_init(&vs.output, "vnc-worker-output");
> -
> vnc_lock_queue(queue);
> while (QTAILQ_EMPTY(&queue->jobs) && !queue->exit) {
> qemu_cond_wait(&queue->cond, &queue->mutex);
Reviewed-by: Daniel P. Berrange <address@hidden>
> --
> 1.8.3.1
>
Regards,
Daniel
--
|: http://berrange.com -o- http://www.flickr.com/photos/dberrange/ :|
|: http://libvirt.org -o- http://virt-manager.org :|
|: http://autobuild.org -o- http://search.cpan.org/~danberr/ :|
|: http://entangle-photo.org -o- http://live.gnome.org/gtk-vnc :|
- Re: [Qemu-devel] [PATCH 14/19] vnc: only alloc server surface with clients connected, (continued)
- [Qemu-devel] [PATCH 16/19] vnc: recycle empty vs->output buffer, Gerd Hoffmann, 2015/10/30
- [Qemu-devel] [PATCH 12/19] vnc: factor out vnc_update_server_surface, Gerd Hoffmann, 2015/10/30
- [Qemu-devel] [PATCH 01/19] buffer: make the Buffer capacity increase in powers of two, Gerd Hoffmann, 2015/10/30
- [Qemu-devel] [PATCH 11/19] vnc: add vnc_width+vnc_height helpers, Gerd Hoffmann, 2015/10/30
- [Qemu-devel] [PATCH 15/19] vnc: fix local state init, Gerd Hoffmann, 2015/10/30
- Re: [Qemu-devel] [PATCH 15/19] vnc: fix local state init,
Daniel P. Berrange <=
- [Qemu-devel] [PATCH 06/19] buffer: add tracing, Gerd Hoffmann, 2015/10/30
- [Qemu-devel] [PATCH 09/19] vnc-jobs: move buffer reset, use new buffer move, Gerd Hoffmann, 2015/10/30
- Re: [Qemu-devel] [PATCH 00/19] buffer/vnc: improve vnc buffer hsndling, Daniel P. Berrange, 2015/10/30