Fix for 32-bit architectures

arl · arl · commit 8e1e8ec367e7 · 2024-12-06T12:40:48.000+01:00
diff --git a/timezones.go b/timezones.go
@@ -94,8 +94,9 @@ func buildTZData(template *Template) ([]byte, error) {
 	if len(template.Zones) == 0 && template.Extend == "" {
 		return nil, fmt.Errorf("either zones or extend string need to be present")
 	}
-	if len(template.Changes) > math.MaxUint32 {
-		return nil, fmt.Errorf("too many changes (%d), max is %d", len(template.Changes), math.MaxUint32)
+	nchanges := int64(len(template.Changes))
+	if nchanges > math.MaxUint32 {
+		return nil, fmt.Errorf("too many changes (%d), max is %v", nchanges, int64(math.MaxUint32))
 	}
 
 	size := headerSize + // v1 header + empty v1 data block

Original file line number	Diff line number	Diff line change
`@@ -94,8 +94,9 @@ func buildTZData(template *Template) ([]byte, error) {`
`94`	`94`	`if len(template.Zones) == 0 && template.Extend == "" {`
`95`	`95`	`return nil, fmt.Errorf("either zones or extend string need to be present")`
`96`	`96`	`}`
`97`		`- if len(template.Changes) > math.MaxUint32 {`
`98`		`- return nil, fmt.Errorf("too many changes (%d), max is %d", len(template.Changes), math.MaxUint32)`
	`97`	`+ nchanges := int64(len(template.Changes))`
	`98`	`+ if nchanges > math.MaxUint32 {`
	`99`	`+ return nil, fmt.Errorf("too many changes (%d), max is %v", nchanges, int64(math.MaxUint32))`
`99`	`100`	`}`
`100`	`101`
`101`	`102`	`size := headerSize + // v1 header + empty v1 data block`