diff options
author | Alberto Garcia <berto@igalia.com> | 2015-04-17 14:52:43 +0300 |
---|---|---|
committer | Kevin Wolf <kwolf@redhat.com> | 2015-04-28 15:36:09 +0200 |
commit | d5a8ee60a0fbc20a2c2d02f3bda1bb1bd365f1ee (patch) | |
tree | 7755e310e852704dbb2022a426d94e23bae02d4b /blockdev.c | |
parent | 9419874f709469de16c1bced7731bfecb07fe1cf (diff) | |
download | qemu-d5a8ee60a0fbc20a2c2d02f3bda1bb1bd365f1ee.tar.gz qemu-d5a8ee60a0fbc20a2c2d02f3bda1bb1bd365f1ee.tar.bz2 qemu-d5a8ee60a0fbc20a2c2d02f3bda1bb1bd365f1ee.zip |
qmp: fill in the image field in BlockDeviceInfo
The image field in BlockDeviceInfo is supposed to contain an ImageInfo
object. However that is being filled in by bdrv_query_info(), not by
bdrv_block_device_info(), which is where BlockDeviceInfo is actually
created.
Anyone calling bdrv_block_device_info() directly will get a null image
field. As a consequence of this, the HMP command 'info block -n -v'
crashes QEMU.
This patch moves the code that fills in that field from
bdrv_query_info() to bdrv_block_device_info().
Signed-off-by: Alberto Garcia <berto@igalia.com>
Message-id: 1429271563-3765-1-git-send-email-berto@igalia.com
Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
Signed-off-by: Kevin Wolf <kwolf@redhat.com>
Diffstat (limited to 'blockdev.c')
-rw-r--r-- | blockdev.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/blockdev.c b/blockdev.c index e67c701879..a136b2ed47 100644 --- a/blockdev.c +++ b/blockdev.c @@ -2392,7 +2392,7 @@ out: BlockDeviceInfoList *qmp_query_named_block_nodes(Error **errp) { - return bdrv_named_nodes_list(); + return bdrv_named_nodes_list(errp); } void qmp_blockdev_backup(const char *device, const char *target, |