Fix out of bounds panic in registery

When registering or updating the registry with provider information that does not have provider addresses or publisher addresses, an out of bounds panic could happen because it was assumed that the provider info whould at least have provider addresses. This is not assumed now.
ipni · Apr 4, 2022 · f333592 · f333592
1 parent 084dd6e
commit f333592
Showing 1 changed file with 6 additions and 1 deletion.
diff --git a/internal/registry/registry.go b/internal/registry/registry.go
@@ -391,7 +391,12 @@ func (r *Registry) RegisterOrUpdate(ctx context.Context, providerID peer.ID, add
 	}
 
 	if info.Publisher.Validate() == nil && info.PublisherAddr == nil && info.Publisher == info.AddrInfo.ID {
-		info.PublisherAddr = info.AddrInfo.Addrs[0]
+		if len(info.AddrInfo.Addrs) == 0 {
+			log.Warnw("Register provider with no provider or publisher addresses",
+				"provider", info.AddrInfo.ID, "publisher", info.Publisher)
+		} else {
+			info.PublisherAddr = info.AddrInfo.Addrs[0]
+		}
 	}
 
 	if len(addrs) != 0 {