codec: fix regression from msgpack container len limit fix which failed compile on 32-bit machines

Fixes #433

Author: ugorji

codec/msgpack.go

@@ -293,7 +293,7 @@ func (e *msgpackEncDriver[T]) writeContainerLen(ct msgpackContainerType, l int)
 	} else if l < 65536 {
 		e.w.writen1(ct.b16)
 		e.w.writen2(bigen.PutUint16(uint16(l)))
-	} else if l <= mpMaxLen {
+	} else if uint(l) <= mpMaxLen {
 		e.w.writen1(ct.b32)
 		e.w.writen4(bigen.PutUint32(uint32(l)))
 	} else {

codec/msgpack.mono.generated.go

@@ -3353,7 +3353,7 @@ func (e *msgpackEncDriverBytes) writeContainerLen(ct msgpackContainerType, l int
 	} else if l < 65536 {
 		e.w.writen1(ct.b16)
 		e.w.writen2(bigen.PutUint16(uint16(l)))
-	} else if l <= mpMaxLen {
+	} else if uint(l) <= mpMaxLen {
 		e.w.writen1(ct.b32)
 		e.w.writen4(bigen.PutUint32(uint32(l)))
 	} else {
@@ -7369,7 +7369,7 @@ func (e *msgpackEncDriverIO) writeContainerLen(ct msgpackContainerType, l int) {
 	} else if l < 65536 {
 		e.w.writen1(ct.b16)
 		e.w.writen2(bigen.PutUint16(uint16(l)))
-	} else if l <= mpMaxLen {
+	} else if uint(l) <= mpMaxLen {
 		e.w.writen1(ct.b32)
 		e.w.writen4(bigen.PutUint32(uint32(l)))
 	} else {

codec/msgpack_test.go

@@ -111,14 +111,15 @@ func doTestMsgpackDecodeMapAndExtSizeMismatch(t *testing.T, h Handle) {
 
 func doTestMsgpackIntOverflow(t *testing.T, h Handle) {
 	defer testSetup(t, &h)()
-	if cpu32Bit {
-		t.Skip("test skipped on 32-bit machine")
-	}
 	if testing.Short() {
 		t.Skip("skipping test in short mode")
 	}
+	if cpu32Bit {
+		t.Skip("test skipped on 32-bit machine")
+	}
 	overflowString := "\xa7input_b\xd9\x12HACKER OVERWRITTEN"
-	var sb = make([]byte, (1<<32)+len(overflowString)+64)
+	var llen uint64 = (1 << 32) + 64 // prevent compile failure
+	var sb = make([]byte, int(llen))
 	copy(sb, overflowString)
 	overflowString = stringView(sb)
 	// sb.Grow((1 << 32) + len(overflowString) + 64)