[PATCH qemu-server v2 0/2] fix #7828: avoid parsing absolute paths as volume ids.

[Thomas Ellmenreich <t.ellmenreich@proxmox.com>]

This series refactors all checks of drive and volume ids as absolute paths
into two separate helper functions, and uses them in all occurrences of such
checks.

The new helper functions are then also applied to make sure that drives that
are absolute paths are not added as unused drives to a vm. Just like is the
case for cdrom drives.

This issue was reported in the community forum: [0]. The issue arose, as
@Daniel Kral mentioned, because of the addition of the config check and parsing
of the volume id with commit: 5b5c5768, which did not consider the possibility
of absolute paths as volume ids.

Changes since v1:

- Prepended an additional patch to introduce the two helper functions, as the
  check for absolute paths was performed a considerable amount of times.

- Use of the created helper functions:

- Addition of the "Fixes 5b..." to the commit message

[0]: https://forum.proxmox.com/threads/possible-regression-in-pve-9-2-x-unable-to-remove-physical-passthrough-disks-using-qm-set-delete.185191/


qemu-manager:

Thomas Ellmenreich (2):
  refactor checks for absolute path as drive
  fix #7828: avoid parsing absolute paths as volume ids

 src/PVE/QemuConfig.pm          |  2 +-
 src/PVE/QemuMigrate.pm         |  2 +-
 src/PVE/QemuServer.pm          | 22 +++++++++-------------
 src/PVE/QemuServer/Blockdev.pm |  4 ++--
 src/PVE/QemuServer/Drive.pm    |  8 +++++++-
 src/PVE/QemuServer/Helpers.pm  |  5 +++++
 6 files changed, 25 insertions(+), 18 deletions(-)


Summary over all repositories:
  6 files changed, 25 insertions(+), 18 deletions(-)

-- 
Generated by murpp 0.12.0