yesBad/pve-storage
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

storage plugin: move get_subdir call

Browse Source 
since list_volumes is only supposed to be called with filtered content
types, this should ensure that get_subdir is only called for plugins
that have a defined 'path' property, like the old code in
PVE::Storage::template_list did.

Signed-off-by: Fabian Grünbichler <f.gruenbichler@proxmox.com>
This commit is contained in:
Fabian Grünbichler
2019-07-19 13:53:45 +02:00
parent c2fc9fbe1d
commit a14e0a5e95
1 changed files with 12 additions and 10 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
PVE/Storage/Plugin.pm
View File

@ -954,11 +954,12 @@ sub list_volumes {
foreach my $ct (@$content_types) { foreach my $ct (@$content_types) {
my $data; my $data;
my $path = $class->get_subdir($scfg, $ct);
if ($ct eq 'images') { if ($ct eq 'images') {
$data = $class->list_images($storeid, $scfg, $vmid); $data = $class->list_images($storeid, $scfg, $vmid);
} elsif ($ct eq 'iso' && !defined($vmid)) { } elsif ($scfg->{path}) {
my $path = $class->get_subdir($scfg, $ct);
if ($ct eq 'iso' && !defined($vmid)) {
$data = $get_subdir_files->($storeid, $path, 'iso'); $data = $get_subdir_files->($storeid, $path, 'iso');
} elsif ($ct eq 'vztmpl'&& !defined($vmid)) { } elsif ($ct eq 'vztmpl'&& !defined($vmid)) {
$data = $get_subdir_files->($storeid, $path, 'vztmpl'); $data = $get_subdir_files->($storeid, $path, 'vztmpl');
@ -967,6 +968,7 @@ sub list_volumes {
} elsif ($ct eq 'snippets') { } elsif ($ct eq 'snippets') {
$data = $get_subdir_files->($storeid, $path, 'snippets'); $data = $get_subdir_files->($storeid, $path, 'snippets');
} }
}
next if !$data; next if !$data;