diff options
author | Ashijeet Acharya <ashijeetacharya@gmail.com> | 2016-09-27 22:23:32 +0530 |
---|---|---|
committer | John Snow <jsnow@redhat.com> | 2016-09-29 15:50:29 -0400 |
commit | ca44141d5fb801dd5903102acefd0f2d8e8bb6a1 (patch) | |
tree | fad957e61614cb179d6fc731af800ba72a07b79b /include/hw/ide | |
parent | c9f7acd57552b919a7b9b1c381383960307c0ada (diff) | |
download | qemu-ca44141d5fb801dd5903102acefd0f2d8e8bb6a1.zip qemu-ca44141d5fb801dd5903102acefd0f2d8e8bb6a1.tar.gz qemu-ca44141d5fb801dd5903102acefd0f2d8e8bb6a1.tar.bz2 |
ide: Fix memory leak in ide_register_restart_cb()
Fix a memory leak in ide_register_restart_cb() in hw/ide/core.c and add
idebus_unrealize() in hw/ide/qdev.c to have calls to
qemu_del_vm_change_state_handler() to deal with the dangling change
state handler during hot-unplugging ide devices which might lead to a
crash.
Signed-off-by: Ashijeet Acharya <ashijeetacharya@gmail.com>
Reviewed-by: John Snow <jsnow@redhat.com>
Message-id: 1474995212-10580-1-git-send-email-ashijeetacharya@gmail.com
[Minor whitespace fix --js]
Signed-off-by: John Snow <jsnow@redhat.com>
Diffstat (limited to 'include/hw/ide')
-rw-r--r-- | include/hw/ide/internal.h | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/include/hw/ide/internal.h b/include/hw/ide/internal.h index a6dd2c3..88dc118 100644 --- a/include/hw/ide/internal.h +++ b/include/hw/ide/internal.h @@ -482,6 +482,7 @@ struct IDEBus { uint32_t retry_nsector; PortioList portio_list; PortioList portio2_list; + VMChangeStateEntry *vmstate; }; #define TYPE_IDE_DEVICE "ide-device" |