api: backup: update: check permissions of delete params too

[pve-manager.git] / PVE / API2 / Backup.pm

diff --git a/PVE/API2/Backup.pm b/PVE/API2/Backup.pm
index c0800bac2b670b5eb227ef16d79bd540ee0b1f14..684a078e785dc6f307725b83f277a20fe7e50f4d 100644 (file)
--- a/PVE/API2/Backup.pm
+++ b/PVE/API2/Backup.pm
@@ -54,6 +54,14 @@ sub assert_param_permission_common {
     }
 }
 
     }
 }
 

+my sub assert_param_permission_update {
+    my ($rpcenv, $user, $update, $delete) = @_;
+    return if $user eq 'root@pam'; # always OK
+
+    assert_param_permission_common($rpcenv, $user, $update);
+    assert_param_permission_common($rpcenv, $user, $delete);
+}
+

 my $convert_to_schedule = sub {
     my ($job) = @_;
 
 my $convert_to_schedule = sub {
     my ($job) = @_;
 


@@ -424,8 +432,6 @@ __PACKAGE__->register_method({
        my $rpcenv = PVE::RPCEnvironment::get();
        my $user = $rpcenv->get_user();
 
        my $rpcenv = PVE::RPCEnvironment::get();
        my $user = $rpcenv->get_user();
 

-       assert_param_permission_common($rpcenv, $user, $param);
-

        if (my $pool = $param->{pool}) {
            $rpcenv->check_pool_exist($pool);
            $rpcenv->check($user, "/pool/$pool", ['VM.Backup']);
        if (my $pool = $param->{pool}) {
            $rpcenv->check_pool_exist($pool);
            $rpcenv->check($user, "/pool/$pool", ['VM.Backup']);


@@ -437,6 +443,8 @@ __PACKAGE__->register_method({
        my $delete = extract_param($param, 'delete');
        $delete = { map { $_ => 1 } PVE::Tools::split_list($delete) } if $delete;
 
        my $delete = extract_param($param, 'delete');
        $delete = { map { $_ => 1 } PVE::Tools::split_list($delete) } if $delete;
 

+       assert_param_permission_update($rpcenv, $user, $param, $delete);
+

        my $update_job = sub {
            my $data = cfs_read_file('vzdump.cron');
            my $jobs_data = cfs_read_file('jobs.cfg');
        my $update_job = sub {
            my $data = cfs_read_file('vzdump.cron');
            my $jobs_data = cfs_read_file('jobs.cfg');