Check for bounds.

konstructio · Nov 12, 2024 · 021f562 · 021f562
1 parent d8e8529
commit 021f562
Showing 1 changed file with 5 additions and 1 deletion.
diff --git a/internal/docker/docker.go b/internal/docker/docker.go
@@ -69,7 +69,11 @@ func (c *Client) RemoveColonyK3sContainer(ctx context.Context) error {
 		return fmt.Errorf("error getting %q container: %w", constants.ColonyK3sContainerName, err)
 	}
 
-	c.log.Infof("found container name %q with ID %q", strings.TrimPrefix(k3scontainer.Names[0], "/"), k3scontainer.ID[:constants.DefaultDockerIDLength])
+	if len(k3scontainer.Names) > 0 && len(k3scontainer.ID) > constants.DefaultDockerIDLength {
+		c.log.Infof("found container name %q with ID %q", strings.TrimPrefix(k3scontainer.Names[0], "/"), k3scontainer.ID[:constants.DefaultDockerIDLength])
+	} else {
+		c.log.Warnf("found container with ID %q -- unable to parse a name or an ID out of it", k3scontainer.ID)
+	}
 
 	err = c.cli.ContainerStop(ctx, k3scontainer.ID, containerTypes.StopOptions{})
 	if err != nil {