[libvirt] [PATCH] qemu: Check address when attaching a virtio disk with an invalid address.
Ján Tomko
jtomko at redhat.com
Wed Sep 2 13:54:49 UTC 2015
On Wed, Sep 02, 2015 at 04:50:50PM +0800, rbian wrote:
> https://bugzilla.redhat.com/show_bug.cgi?id=1257844
>
> Attach-device can hotplug a virtio disk device with any address now.
> It need to validate the address before the attachment. This patch
> fix the problem.
>
> Signed-off-by: rbian <rbian at redhat.com>
We prefer using full names for authors, that is, at least two words with
capital letters separated by a space.
> ---
> src/qemu/qemu_hotplug.c | 17 +++++++++++++++++
> 1 file changed, 17 insertions(+)
>
> diff --git a/src/qemu/qemu_hotplug.c b/src/qemu/qemu_hotplug.c
> index e71a204..be24993 100644
> --- a/src/qemu/qemu_hotplug.c
> +++ b/src/qemu/qemu_hotplug.c
> @@ -331,6 +331,23 @@ qemuDomainAttachVirtioDiskDevice(virConnectPtr conn,
> disk->info.type = VIR_DOMAIN_DEVICE_ADDRESS_TYPE_CCW;
> else if (virQEMUCapsGet(priv->qemuCaps, QEMU_CAPS_VIRTIO_S390))
> disk->info.type = VIR_DOMAIN_DEVICE_ADDRESS_TYPE_VIRTIO_S390;
> + } else {
> + if (STREQLEN(vm->def->os.machine, "s390-ccw", 8) &&
> + virQEMUCapsGet(priv->qemuCaps, QEMU_CAPS_VIRTIO_CCW)) {
> + if (!virDomainDeviceAddressIsValid(&disk->info,
> + VIR_DOMAIN_DEVICE_ADDRESS_TYPE_CCW)) {
> + virReportError(VIR_ERR_OPERATION_FAILED, "%s",
> + _("device cannot be attached without a valid CCW address"));
> + goto error;
> + }
This part is covered by the patch posted by John.
> + } else {
> + if (!virDomainDeviceAddressIsValid(&disk->info,
> + VIR_DOMAIN_DEVICE_ADDRESS_TYPE_PCI)) {
> + virReportError(VIR_ERR_OPERATION_FAILED, "%s",
> + _("device cannot be attached without a valid PCI address"));
> + goto error;
> + }
> + }
It looks like this would break hotplugging devices with S390 address
types.
We should only check it when the type is VIR_DOMAIN_DEVICE_ADDRESS_TYPE_PCI.
Are there other devices where we don't check the validity of the
user-supplied address?
Jan
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20150902/d09a7d4b/attachment-0001.sig>
More information about the libvir-list
mailing list