[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [6911] char: Fix closing of various char devices (Jan Kiszk
From: |
Anthony Liguori |
Subject: |
[Qemu-devel] [6911] char: Fix closing of various char devices (Jan Kiszka) |
Date: |
Sat, 28 Mar 2009 17:58:14 +0000 |
Revision: 6911
http://svn.sv.gnu.org/viewvc/?view=rev&root=qemu&revision=6911
Author: aliguori
Date: 2009-03-28 17:58:14 +0000 (Sat, 28 Mar 2009)
Log Message:
-----------
char: Fix closing of various char devices (Jan Kiszka)
This patch fixes several issues around closing char devices. Affected
were pty (timer was left behind, even running), udp (no close handling
at all) and tcp (missing async IO handler cleanup). The bugs either
caused segfaults or stalled the qemu process. So far, hot-unplugging USB
serial adapters suffered from this.
Signed-off-by: Jan Kiszka <address@hidden>
Signed-off-by: Anthony Liguori <address@hidden>
Modified Paths:
--------------
trunk/qemu-char.c
Modified: trunk/qemu-char.c
===================================================================
--- trunk/qemu-char.c 2009-03-28 17:55:19 UTC (rev 6910)
+++ trunk/qemu-char.c 2009-03-28 17:58:14 UTC (rev 6911)
@@ -929,6 +929,8 @@
qemu_set_fd_handler2(s->fd, NULL, NULL, NULL, NULL);
close(s->fd);
+ qemu_del_timer(s->timer);
+ qemu_free_timer(s->timer);
qemu_free(s);
}
@@ -1758,6 +1760,16 @@
}
}
+static void udp_chr_close(CharDriverState *chr)
+{
+ NetCharDriver *s = chr->opaque;
+ if (s->fd >= 0) {
+ qemu_set_fd_handler(s->fd, NULL, NULL, NULL);
+ closesocket(s->fd);
+ }
+ qemu_free(s);
+}
+
static CharDriverState *qemu_chr_open_udp(const char *def)
{
CharDriverState *chr = NULL;
@@ -1791,6 +1803,7 @@
chr->opaque = s;
chr->chr_write = udp_chr_write;
chr->chr_update_read_handler = udp_chr_update_read_handler;
+ chr->chr_close = udp_chr_close;
return chr;
return_err:
@@ -1993,10 +2006,14 @@
static void tcp_chr_close(CharDriverState *chr)
{
TCPCharDriver *s = chr->opaque;
- if (s->fd >= 0)
+ if (s->fd >= 0) {
+ qemu_set_fd_handler(s->fd, NULL, NULL, NULL);
closesocket(s->fd);
- if (s->listen_fd >= 0)
+ }
+ if (s->listen_fd >= 0) {
+ qemu_set_fd_handler(s->listen_fd, NULL, NULL, NULL);
closesocket(s->listen_fd);
+ }
qemu_free(s);
}
[Prev in Thread] |
Current Thread |
[Next in Thread] |
- [Qemu-devel] [6911] char: Fix closing of various char devices (Jan Kiszka),
Anthony Liguori <=