Re: [Qemu-devel] [PATCH] vnc_refresh: calling vnc_update_client might free vs

2010-01-27 Thread Anthony Liguori
On 01/27/2010 06:29 AM, Stefano Stabellini wrote: On Wed, 27 Jan 2010, Gerd Hoffmann wrote: On 01/27/10 01:07, Anthony Liguori wrote: On 01/25/2010 06:54 AM, Stefano Stabellini wrote: Hi all, this patch fixes another bug in vnc_refresh: calling vnc_update_client might cause v

Re: [Qemu-devel] [PATCH] vnc_refresh: calling vnc_update_client might free vs

2010-01-27 Thread Stefano Stabellini
On Wed, 27 Jan 2010, Gerd Hoffmann wrote: > On 01/27/10 01:07, Anthony Liguori wrote: > > On 01/25/2010 06:54 AM, Stefano Stabellini wrote: > >> Hi all, > >> this patch fixes another bug in vnc_refresh: calling vnc_update_client > >> might cause vs to be free()ed, in this case we cannot access vs->

Re: [Qemu-devel] [PATCH] vnc_refresh: calling vnc_update_client might free vs

2010-01-27 Thread Gerd Hoffmann
On 01/27/10 01:07, Anthony Liguori wrote: On 01/25/2010 06:54 AM, Stefano Stabellini wrote: Hi all, this patch fixes another bug in vnc_refresh: calling vnc_update_client might cause vs to be free()ed, in this case we cannot access vs->next right after to examine the next item on the list. Sign

Re: [Qemu-devel] [PATCH] vnc_refresh: calling vnc_update_client might free vs

2010-01-26 Thread Anthony Liguori
On 01/25/2010 06:54 AM, Stefano Stabellini wrote: Hi all, this patch fixes another bug in vnc_refresh: calling vnc_update_client might cause vs to be free()ed, in this case we cannot access vs->next right after to examine the next item on the list. Signed-off-by: Stefano Stabellini Applied

[Qemu-devel] [PATCH] vnc_refresh: calling vnc_update_client might free vs

2010-01-25 Thread Stefano Stabellini
Hi all, this patch fixes another bug in vnc_refresh: calling vnc_update_client might cause vs to be free()ed, in this case we cannot access vs->next right after to examine the next item on the list. Signed-off-by: Stefano Stabellini --- diff --git a/vnc.c b/vnc.c index cc2a26e..92facde 100644 -