From 5eaa38e58178a37e979f41b876c3ea28b3cd10b8 Mon Sep 17 00:00:00 2001
From: Matthew R Kasun <mkasun@nusak.ca>
Date: Fri, 11 Aug 2023 18:10:57 -0400
Subject: [PATCH] guard against blank address or address6

---
 controllers/migrate.go | 22 +++++++++++++++-------
 1 file changed, 15 insertions(+), 7 deletions(-)

diff --git a/controllers/migrate.go b/controllers/migrate.go
index d8b54062d..fa03fccf9 100644
--- a/controllers/migrate.go
+++ b/controllers/migrate.go
@@ -158,27 +158,35 @@ func convertLegacyNode(legacy models.LegacyNode, hostID uuid.UUID) models.Node {
 	node.ID, _ = uuid.Parse(legacy.ID)
 	node.HostID = hostID
 	node.Network = legacy.Network
+	valid4 := true
+	valid6 := true
 	_, cidr4, err := net.ParseCIDR(legacy.NetworkSettings.AddressRange)
 	if err != nil {
+		valid4 = false
 		slog.Warn("parsing address range", "error", err)
 	} else {
 		node.NetworkRange = *cidr4
 	}
 	_, cidr6, err := net.ParseCIDR(legacy.NetworkSettings.AddressRange6)
 	if err != nil {
-		slog.Warn("paring address range6", "error", err)
+		valid6 = false
+		slog.Warn("parsing address range6", "error", err)
 	} else {
 		node.NetworkRange6 = *cidr6
 	}
 	node.Server = servercfg.GetServer()
 	node.Connected = models.ParseBool(legacy.Connected)
-	node.Address = net.IPNet{
-		IP:   net.ParseIP(legacy.Address),
-		Mask: cidr4.Mask,
+	if valid4 {
+		node.Address = net.IPNet{
+			IP:   net.ParseIP(legacy.Address),
+			Mask: cidr4.Mask,
+		}
 	}
-	node.Address6 = net.IPNet{
-		IP:   net.ParseIP(legacy.Address6),
-		Mask: cidr6.Mask,
+	if valid6 {
+		node.Address6 = net.IPNet{
+			IP:   net.ParseIP(legacy.Address6),
+			Mask: cidr6.Mask,
+		}
 	}
 	node.Action = models.NODE_NOOP
 	node.LocalAddress = net.IPNet{