summary refs log tree commit diff stats
path: root/ui/spice-core.c
diff options
context:
space:
mode:
authorStefan Hajnoczi <stefanha@redhat.com>2012-12-19 14:07:16 +0100
committerStefan Hajnoczi <stefanha@redhat.com>2013-01-04 14:38:05 +0100
commitbfb82a28752d29291adf932c3a9941e8383203af (patch)
tree9047e893e9d7a929cf0ffab4bcbc86e094961e8d /ui/spice-core.c
parent586b0bef84c202bb2256a35eb71bfd6670262bd7 (diff)
downloadfocaccia-qemu-bfb82a28752d29291adf932c3a9941e8383203af.tar.gz
focaccia-qemu-bfb82a28752d29291adf932c3a9941e8383203af.zip
spice: drop incorrect vm_change_state_handler() opaque
The spice_server pointer is a global variable and
vm_change_state_handler() therefore does not use its opaque parameter.

The vm change state handler is added with a pointer to the spice_server
pointer.  This is useless and we probably would not want 2 levels of
pointers.

Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
Reviewed-by: Uri Lublin <uril@redhat.com>
Diffstat (limited to 'ui/spice-core.c')
-rw-r--r--ui/spice-core.c5
1 files changed, 2 insertions, 3 deletions
diff --git a/ui/spice-core.c b/ui/spice-core.c
index 3e44779107..d83de2a46e 100644
--- a/ui/spice-core.c
+++ b/ui/spice-core.c
@@ -709,7 +709,7 @@ void qemu_spice_init(void)
     qemu_spice_input_init();
     qemu_spice_audio_init();
 
-    qemu_add_vm_change_state_handler(vm_change_state_handler, &spice_server);
+    qemu_add_vm_change_state_handler(vm_change_state_handler, NULL);
 
     g_free(x509_key_file);
     g_free(x509_cert_file);
@@ -736,8 +736,7 @@ int qemu_spice_add_interface(SpiceBaseInstance *sin)
          */
         spice_server = spice_server_new();
         spice_server_init(spice_server, &core_interface);
-        qemu_add_vm_change_state_handler(vm_change_state_handler,
-                                         &spice_server);
+        qemu_add_vm_change_state_handler(vm_change_state_handler, NULL);
     }
 
     return spice_server_add_interface(spice_server, sin);