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

Dir storage creation: check for a sane path

Browse Source 
Ideally we don't need this, but this with the directory
storage this is a user-input field which gets returned
by the storage's path() method which is used in various
external command calls.
This commit is contained in:
Wolfgang Bumiller
2016-06-09 16:54:49 +02:00
committed by Dietmar Maurer
parent 602eacfe6a
commit 5c95e48479
1 changed files with 9 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

9
PVE/Storage/DirPlugin.pm
View File

@ -62,5 +62,14 @@ sub activate_storage {
$class->SUPER::activate_storage($storeid, $scfg, $cache); $class->SUPER::activate_storage($storeid, $scfg, $cache);
} }
sub check_config {
my ($self, $sectionId, $config, $create, $skipSchemaCheck) = @_;
my $opts = PVE::SectionConfig::check_config($self, $sectionId, $config, $create, $skipSchemaCheck);
return $opts if !$create;
if ($opts->{path} !~ m@^/[-/a-zA-Z0-9_.]+$@) {
die "illegal path for directory storage: $opts->{path}\n";
}
return $opts;
}
1; 1;