diff options
| author | Stefan Hajnoczi <stefanha@redhat.com> | 2012-12-19 14:07:16 +0100 |
|---|---|---|
| committer | Stefan Hajnoczi <stefanha@redhat.com> | 2013-01-04 14:38:05 +0100 |
| commit | bfb82a28752d29291adf932c3a9941e8383203af (patch) | |
| tree | 9047e893e9d7a929cf0ffab4bcbc86e094961e8d | |
| parent | 586b0bef84c202bb2256a35eb71bfd6670262bd7 (diff) | |
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>
| -rw-r--r-- | ui/spice-core.c | 5 |
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); |