[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PATCH 1/2] vnc: fix segfault due to failed handshake
From: |
Tim Hardeck |
Subject: |
[Qemu-devel] [PATCH 1/2] vnc: fix segfault due to failed handshake |
Date: |
Tue, 9 Oct 2012 17:21:40 +0200 |
When the VNC server disconnects due to a failed handshake we don't have
vs->bh allocated yet.
Check for this case and don't delete it.
Signed-off-by: Tim Hardeck <address@hidden>
---
ui/vnc.c | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/ui/vnc.c b/ui/vnc.c
index 01b2daf..656895a 100644
--- a/ui/vnc.c
+++ b/ui/vnc.c
@@ -1055,7 +1055,9 @@ static void vnc_disconnect_finish(VncState *vs)
vnc_unlock_output(vs);
qemu_mutex_destroy(&vs->output_mutex);
- qemu_bh_delete(vs->bh);
+ if (vs->bh != NULL) {
+ qemu_bh_delete(vs->bh);
+ }
buffer_free(&vs->jobs_buffer);
for (i = 0; i < VNC_STAT_ROWS; ++i) {
--
1.7.10.4
- [Qemu-devel] [PATCH 0/2] fix segfaults triggered by failed vnc handshakes, Tim Hardeck, 2012/10/09
- [Qemu-devel] [PATCH 0/2] fix segfaults triggered by failed vnc handshakes, Tim Hardeck, 2012/10/14
- [Qemu-devel] [PATCH 2/2] qemu queue: fix uninitialized removals, Tim Hardeck, 2012/10/14
- Re: [Qemu-devel] [PATCH 2/2] qemu queue: fix uninitialized removals, Andreas Färber, 2012/10/17
- Re: [Qemu-devel] [PATCH 2/2] qemu queue: fix uninitialized removals, Tim Hardeck, 2012/10/17
- Re: [Qemu-devel] [PATCH 2/2] qemu queue: fix uninitialized removals, Kevin Wolf, 2012/10/18
- Re: [Qemu-devel] [PATCH 2/2] qemu queue: fix uninitialized removals, Andreas Färber, 2012/10/18
- Re: [Qemu-devel] [PATCH 2/2] qemu queue: fix uninitialized removals, Peter Maydell, 2012/10/18
- Re: [Qemu-devel] [PATCH 2/2] qemu queue: fix uninitialized removals, Peter Maydell, 2012/10/18
[Qemu-devel] [PATCH 1/2] vnc: fix segfault due to failed handshake, Tim Hardeck, 2012/10/14