From: Stefan Hanreich <s.hanreich@proxmox.com>
Date: Wed, 22 Nov 2023 12:27:28 +0000 (+0100)
Subject: subnets: only delete macs.db entries if mac is available
X-Git-Url: https://git.puffer.fish/?a=commitdiff_plain;h=5aea20cd5b22edf1128861a4f326bc4ac7f46145;p=matthieu%2Fpve-network.git

subnets: only delete macs.db entries if mac is available

When removing a gateway do not attempt to delete its entry from
macs.db since we do not have anything cached for the gateway anyway.

Reported-By: Wolfgang Bumiller <w.bumiller@proxmox.com>
Signed-off-by: Stefan Hanreich <s.hanreich@proxmox.com>
---

diff --git a/src/PVE/Network/SDN/Subnets.pm b/src/PVE/Network/SDN/Subnets.pm
index 8e2a6aa..05d2de2 100644
--- a/src/PVE/Network/SDN/Subnets.pm
+++ b/src/PVE/Network/SDN/Subnets.pm
@@ -399,7 +399,7 @@ sub del_ip {
 	my $plugin = PVE::Network::SDN::Ipams::Plugin->lookup($plugin_config->{type});
 	$plugin->del_ip($plugin_config, $subnetid, $subnet, $ip);
 
-	eval { PVE::Network::SDN::Ipams::del_cache_mac_ip($mac, $ip); };
+	eval { PVE::Network::SDN::Ipams::del_cache_mac_ip($mac, $ip) if $mac; };
 	warn $@ if $@;
     }