fix: Prevent panic when merging IPAM config

Signed-off-by: Suleiman Dibirov <idsulik@gmail.com>

Author: idsulik

override/merge.go

@@ -226,6 +226,13 @@ func mergeUlimit(_ any, o any, p tree.Path) (any, error) {
 }
 
 func mergeIPAMConfig(c any, o any, path tree.Path) (any, error) {
+	if _, ok := c.([]any); !ok {
+		return nil, fmt.Errorf("cannot merge IPAM config at %s: base value is not a list", path)
+	}
+	if _, ok := o.([]any); !ok {
+		return nil, fmt.Errorf("cannot merge IPAM config at %s: override value is not a list", path)
+	}
+
 	var ipamConfigs []any
 	for _, original := range c.([]any) {
 		right := convertIntoMapping(original, nil)

override/merge_networks_test.go

@@ -17,7 +17,10 @@
 package override
 
 import (
+	"strings"
 	"testing"
+
+	"github.com/compose-spec/compose-go/v2/tree"
 )
 
 func Test_mergeYamlServiceNetworkSequence(t *testing.T) {
@@ -226,3 +229,40 @@ networks:
   network2:
 `)
 }
+
+func Test_mergeIPAMConfig_invalidTypes(t *testing.T) {
+	tests := []struct {
+		name    string
+		c       any
+		o       any
+		wantErr string
+	}{
+		{
+			name:    "c is not a list",
+			c:       map[string]any{},
+			o:       []any{},
+			wantErr: "base value is not a list",
+		},
+		{
+			name:    "o is not a list",
+			c:       []any{},
+			o:       map[string]any{},
+			wantErr: "override value is not a list",
+		},
+		{
+			name:    "both are not lists",
+			c:       "invalid",
+			o:       123,
+			wantErr: "base value is not a list",
+		},
+	}
+
+	for _, tt := range tests {
+		t.Run(tt.name, func(t *testing.T) {
+			_, err := mergeIPAMConfig(tt.c, tt.o, tree.NewPath("networks", "ipam", "config"))
+			if err == nil || !strings.Contains(err.Error(), tt.wantErr) {
+				t.Fatalf("expected error containing %q, got %v", tt.wantErr, err)
+			}
+		})
+	}
+}