[PATCH 0/6] qemu: Fix crash when validating a XML without disk's <target>

Peter Krempa pkrempa at redhat.com
Tue Sep 7 13:22:56 UTC 2021


Apart from fixing the crash the validation code is fixed to do the
correct thing and a test case is added.

Peter Krempa (6):
  qemuDomainDefValidateDiskLunSource: Unbreak error messages
  conf: validate: Move qemu-specific LUN disk validation to global
    validation
  conf: validate: Run global device definition validation before
    callbacks
  conf: Don't call 'virDomainDiskDefAssignAddress' when disk->dst is
    NULL
  virDomainDiskDefValidate: Move validation of disk target
  qemuxml2argvtest: Add test case for missing disk '<target>'

 src/conf/domain_conf.c                        |  1 +
 src/conf/domain_validate.c                    | 94 ++++++++++++++-----
 src/conf/domain_validate.h                    |  2 +
 src/libvirt_private.syms                      |  1 +
 src/qemu/qemu_domain.c                        | 50 ----------
 src/qemu/qemu_domain.h                        |  3 -
 src/qemu/qemu_driver.c                        |  2 +-
 src/qemu/qemu_validate.c                      |  3 -
 .../default-video-type-x86_64-caps-test-0.err |  2 +-
 .../disk-fdc-incompatible-address.err         |  2 +-
 .../disk-ide-incompatible-address.err         |  2 +-
 .../disk-missing-target-invalid.err           |  1 +
 .../disk-missing-target-invalid.xml           | 22 +++++
 .../disk-sata-incompatible-address.err        |  2 +-
 .../disk-scsi-incompatible-address.err        |  2 +-
 .../pseries-default-phb-numa-node.err         |  2 +-
 .../pseries-phb-invalid-target-index-1.err    |  2 +-
 .../pseries-phb-invalid-target-index-2.err    |  2 +-
 .../pseries-phb-invalid-target-index-3.err    |  2 +-
 .../video-invalid-multiple-devices.err        |  2 +-
 ...splay-device-pci-address.x86_64-latest.err |  2 +-
 tests/qemuxml2argvtest.c                      |  1 +
 22 files changed, 109 insertions(+), 93 deletions(-)
 create mode 100644 tests/qemuxml2argvdata/disk-missing-target-invalid.err
 create mode 100644 tests/qemuxml2argvdata/disk-missing-target-invalid.xml

-- 
2.31.1




More information about the libvir-list mailing list