fix use of uninitialized value

$param->{rename} was not checked for definedness even though it is
optional. This lead to a 'use of uninitialized value' when just updating
the cidr.

Signed-off-by: Mira Limbeck <m.limbeck@proxmox.com>
(cherry picked from commit 13d39f198ad142a013077f4b3cc436257f7a1a11)
Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

diff --git a/src/PVE/API2/Firewall/Aliases.pm b/src/PVE/API2/Firewall/Aliases.pm
index 6f421fbe271ff8c75e09fe5533f95b71bd94f0c3..b81dc9d06405417bb55640cb986d34091136287b 100644 (file)
--- a/src/PVE/API2/Firewall/Aliases.pm
+++ b/src/PVE/API2/Firewall/Aliases.pm
@@ -230,7 +230,8 @@ sub register_update_alias {
 
            $aliases->{$name} = $data;
 
-           my $rename = lc($param->{rename});
+           my $rename = $param->{rename};
+           $rename = lc($rename) if $rename;
 
            if ($rename && ($name ne $rename)) {
                raise_param_exc({ name => "alias '$param->{rename}' already exists" })