diff options
| author | Markus Armbruster <armbru@redhat.com> | 2013-11-28 17:02:24 +0100 |
|---|---|---|
| committer | Andreas Färber <afaerber@suse.de> | 2013-12-24 17:26:58 +0100 |
| commit | 061e84f7a469ad1f94f3b5f6a5361b346ab990e8 (patch) | |
| tree | d3142d1c66b0497a1e5f9a7a217d004c557985c3 /qdev-monitor.c | |
| parent | 7ea5e78f3d8d64e99c4017ea211b7518f2629756 (diff) | |
| download | qemu-061e84f7a469ad1f94f3b5f6a5361b346ab990e8.zip qemu-061e84f7a469ad1f94f3b5f6a5361b346ab990e8.tar.gz qemu-061e84f7a469ad1f94f3b5f6a5361b346ab990e8.tar.bz2 | |
qdev-monitor: Avoid device_add crashing on non-device driver name
Watch this:
$ upstream-qemu -nodefaults -S -display none -monitor stdio
QEMU 1.7.50 monitor - type 'help' for more information
(qemu) device_add rng-egd
/work/armbru/qemu/qdev-monitor.c:491:qdev_device_add: Object 0x2089b00 is not an instance of type device
Aborted (core dumped)
Crashes because "rng-egd" exists, but isn't a subtype of TYPE_DEVICE.
Broken in commit 18b6dad.
Cc: qemu-stable@nongnu.org
Signed-off-by: Markus Armbruster <armbru@redhat.com>
Signed-off-by: Andreas Färber <afaerber@suse.de>
Diffstat (limited to 'qdev-monitor.c')
| -rw-r--r-- | qdev-monitor.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/qdev-monitor.c b/qdev-monitor.c index 7877d01..177b849 100644 --- a/qdev-monitor.c +++ b/qdev-monitor.c @@ -478,7 +478,7 @@ DeviceState *qdev_device_add(QemuOpts *opts) } } - if (!oc) { + if (!object_class_dynamic_cast(oc, TYPE_DEVICE)) { qerror_report(QERR_INVALID_PARAMETER_VALUE, "driver", "device type"); return NULL; } |