diff options
| author | Thomas Huth <thuth@redhat.com> | 2019-04-07 11:23:14 +0200 |
|---|---|---|
| committer | Paolo Bonzini <pbonzini@redhat.com> | 2019-04-09 18:34:21 +0200 |
| commit | ae909496e9d33f8c074f1063597298ba1d183133 (patch) | |
| tree | 6cc6c6ea07b98b0ebe1f0932d4b1b3cf34d14531 /hmp.h | |
| parent | 77b17570900fdfff32c5abb8d92fb527e4a0737a (diff) | |
| download | qemu-ae909496e9d33f8c074f1063597298ba1d183133.zip qemu-ae909496e9d33f8c074f1063597298ba1d183133.tar.gz qemu-ae909496e9d33f8c074f1063597298ba1d183133.tar.bz2 | |
hw/i386/pc: Fix crash when hot-plugging nvdimm on older machine types
QEMU currently crashes when you try to hot-plug an "nvdimm" device
on older machine types:
$ qemu-system-x86_64 -monitor stdio -M pc-1.1
QEMU 3.1.92 monitor - type 'help' for more information
(qemu) device_add nvdimm,id=nvdimmn1
qemu-system-x86_64: /home/thuth/devel/qemu/util/error.c:57: error_setv:
Assertion `*errp == ((void *)0)' failed.
Aborted (core dumped)
The call to hotplug_handler_pre_plug() in pc_memory_pre_plug() has been
added recently before the check whether nvdimm is enabled. It should
be done after the check. And while we're at it, also check the errp
after the hotplug_handler_pre_plug(), otherwise errors are silently
ignored here.
Fixes: 9040e6dfa8c3fed87695a3de555d2c775727bb51
Signed-off-by: Thomas Huth <thuth@redhat.com>
Message-Id: <20190407092314.11066-1-thuth@redhat.com>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
Diffstat (limited to 'hmp.h')
0 files changed, 0 insertions, 0 deletions