accounts/abi: return error on fixed bytes with size larger than 32 by…

…tes (#26075) (#677) commit ethereum/go-ethereum@8578eb2. * fixed bytes with size larger than 32 bytes is not allowed * add testcase Reported-by: eugenioclrc <[email protected]> Co-authored-by: zhiqiangxu <[email protected]>
axieinfinity · Feb 13, 2025 · 08ef4d1 · 08ef4d1
1 parent 3d30c05
commit 08ef4d1
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 0 deletions.
diff --git a/accounts/abi/type.go b/accounts/abi/type.go
@@ -154,6 +154,9 @@ func NewType(t string, internalType string, components []ArgumentMarshaling) (ty
 		if varSize == 0 {
 			typ.T = BytesTy
 		} else {
+			if varSize > 32 {
+				return Type{}, fmt.Errorf("unsupported arg type: %s", t)
+			}
 			typ.T = FixedBytesTy
 			typ.Size = varSize
 		}

diff --git a/accounts/abi/type_test.go b/accounts/abi/type_test.go
@@ -366,3 +366,10 @@ func TestGetTypeSize(t *testing.T) {
 		}
 	}
 }
+
+func TestNewFixedBytesOver32(t *testing.T) {
+	_, err := NewType("bytes4096", "", nil)
+	if err == nil {
+		t.Errorf("fixed bytes with size over 32 is not spec'd")
+	}
+}