aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorLuiz Capitulino <lcapitulino@redhat.com>2010-01-08 16:45:53 -0200
committerAnthony Liguori <aliguori@us.ibm.com>2010-01-12 13:24:52 -0600
commit3d72f9a2be640f368229d579e80ce4ef2a823e49 (patch)
tree41ad63128a49ddb0a2edfaff8a7998172dca4901
parent514d97dea6f5ab8b995275c9b022af85a8846e08 (diff)
downloadqemu-3d72f9a2be640f368229d579e80ce4ef2a823e49.zip
qemu-3d72f9a2be640f368229d579e80ce4ef2a823e49.tar.gz
qemu-3d72f9a2be640f368229d579e80ce4ef2a823e49.tar.bz2
QMP: Don't free async event's 'data'
The monitor_protocol_event() function will free the event's data, this is wrong as 'data' management is up to the caller. Signed-off-by: Luiz Capitulino <lcapitulino@redhat.com> Signed-off-by: Anthony Liguori <aliguori@us.ibm.com>
-rw-r--r--monitor.c4
1 files changed, 3 insertions, 1 deletions
diff --git a/monitor.c b/monitor.c
index 3af1d5c..2403a97 100644
--- a/monitor.c
+++ b/monitor.c
@@ -365,8 +365,10 @@ void monitor_protocol_event(MonitorEvent event, QObject *data)
qmp = qdict_new();
timestamp_put(qmp);
qdict_put(qmp, "event", qstring_from_str(event_name));
- if (data)
+ if (data) {
+ qobject_incref(data);
qdict_put_obj(qmp, "data", data);
+ }
monitor_json_emitter(mon, QOBJECT(qmp));
QDECREF(qmp);