From: Thomas Lamprecht <t.lamprecht@proxmox.com>
Date: Wed, 7 Jun 2023 07:33:16 +0000 (+0200)
Subject: api: realm sync: die early if no job options
X-Git-Url: https://git.proxmox.com/?p=pve-access-control.git;a=commitdiff_plain;h=37976acc6d38187be323b983aa56ee3ffa91abe7

api: realm sync: die early if no job options

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
---

diff --git a/src/PVE/API2/AccessControl/RealmSync.pm b/src/PVE/API2/AccessControl/RealmSync.pm
index f5334b8..e5af8b3 100644
--- a/src/PVE/API2/AccessControl/RealmSync.pm
+++ b/src/PVE/API2/AccessControl/RealmSync.pm
@@ -218,11 +218,11 @@ __PACKAGE__->register_method({
 	my $delete = extract_param($param, 'delete');
 	$delete = [PVE::Tools::split_list($delete)] if $delete;
 
+	die "no job options specified\n" if !scalar(keys %$param);
+
 	cfs_lock_file('jobs.cfg', undef, sub {
 	    my $jobs = cfs_read_file('jobs.cfg');
 
-	    die "no options specified\n" if !scalar(keys %$param);
-
 	    my $plugin = PVE::Job::Registry->lookup('realm-sync');
 	    my $opts = $plugin->check_config($id, $param, 0, 1);