X-Git-Url: https://git.proxmox.com/?a=blobdiff_plain;f=src%2FPVE%2FAPI2%2FFirewall%2FGroups.pm;h=5577d43321ad951879e998dbc0d10fe5b958a007;hb=d72c631cacc2112f1ace652adf5d0283ac087b37;hp=8b65f215e6b7eea5acb65d6c72136e98f7f5edef;hpb=ddf1e07dce58c742826f89811396050e78f07e72;p=pve-firewall.git

diff --git a/src/PVE/API2/Firewall/Groups.pm b/src/PVE/API2/Firewall/Groups.pm
index 8b65f21..5577d43 100644
--- a/src/PVE/API2/Firewall/Groups.pm
+++ b/src/PVE/API2/Firewall/Groups.pm
@@ -89,9 +89,11 @@ __PACKAGE__->register_method({
 
 	PVE::Tools::assert_if_modified($digest, $param->{digest});
 
-	foreach my $name (keys %{$cluster_conf->{groups}}) {
-	    raise_param_exc({ name => "Security group '$name' already exists" }) 
-		if !$param->{rename} && $name eq $param->{name};
+	if (!$param->{rename}) {	
+	    foreach my $name (keys %{$cluster_conf->{groups}}) {
+		raise_param_exc({ name => "Security group '$name' already exists" }) 
+		    if $name eq $param->{name};
+	    }
 	}
 
 	if ($param->{rename}) {
@@ -124,7 +126,7 @@ __PACKAGE__->register_method({
 	properties => { 
 	    name => get_standard_option('pve-security-group-name'),
 	    digest => get_standard_option('pve-config-digest'),
-	}
+	},
     },
     returns => { type => 'null' },
     code => sub {