[pve-devel] [PATCH storage 13/26] plugin, btrfs: update list_images and list_volumes
Fiona Ebner
f.ebner at proxmox.com
Wed Jul 30 10:41:57 CEST 2025
Am 30.07.25 um 10:37 AM schrieb Fabian Grünbichler:
> On July 29, 2025 1:15 pm, Wolfgang Bumiller wrote:
>> +sub foreach_guest_file : prototype($$) {
>
> should this go into PVE::Storage::Common?
>
>> + my ($dir, $code) = @_;
>> +
>> + $MAX_VMID = get_standard_option("pve-vmid")->{maximum} if !defined($MAX_VMID);
>
>> +
>> + dir_glob_foreach(
>> + $dir,
>> + qr/\d+/,
>> + sub {
>> + my ($vmid) = @_;
>> + $vmid = int($vmid);
>> + return if $vmid < 100 || $vmid > $MAX_VMID;
>
> this now means list_images lists less than before - somebody might have
> used a very high ID as a placeholder not realizing its outside the "allowed" range..
>
> not sure it's worth it to have this restriction here..
Or a low ID as placeholder ;)
https://bugzilla.proxmox.com/show_bug.cgi?id=6551#c5
More information about the pve-devel
mailing list