d4cdad5
From fad276489cbc04f228d52b7019bee9e7a88c8a86 Mon Sep 17 00:00:00 2001
d4cdad5
From: Amit Shah <amit.shah@redhat.com>
d4cdad5
Date: Fri, 2 Dec 2011 15:42:55 +0530
d4cdad5
Subject: [PATCH 118/118] char: Disable write callback if throttled chardev is
d4cdad5
 detached
d4cdad5
d4cdad5
If a throttled chardev is detached from the frontend device, all future
d4cdad5
callbacks should be suppressed.  Not doing this results in a segfault.
d4cdad5
d4cdad5
Bugzilla: 745758
d4cdad5
Upstream: Not applicable, since throttling is a RHEL6-only feature.
d4cdad5
d4cdad5
Signed-off-by: Amit Shah <amit.shah@redhat.com>
d4cdad5
---
d4cdad5
 qemu-char.c |    5 +++++
d4cdad5
 1 files changed, 5 insertions(+), 0 deletions(-)
d4cdad5
d4cdad5
diff --git a/qemu-char.c b/qemu-char.c
d4cdad5
index 5f67652..5a94919 100644
d4cdad5
--- a/qemu-char.c
d4cdad5
+++ b/qemu-char.c
d4cdad5
@@ -220,6 +220,11 @@ void qemu_chr_add_handlers(CharDriverState *s,
d4cdad5
         ++s->avail_connections;
d4cdad5
     }
d4cdad5
     if (!handlers) {
d4cdad5
+        if (s->write_blocked) {
d4cdad5
+            /* Ensure we disable the callback if we were throttled */
d4cdad5
+            s->chr_disable_write_fd_handler(s);
d4cdad5
+            /* s->write_blocked is cleared below */
d4cdad5
+        }
d4cdad5
         handlers = &null_handlers;
d4cdad5
     }
d4cdad5
     s->chr_can_read = handlers->fd_can_read;
d4cdad5
-- 
d4cdad5
1.7.7.5
d4cdad5