core/types: sanity check the basefee length inside a header (#23171)

karalabe · web-flow · commit e13d14e6a31d · 2021-07-06T22:02:38.000+02:00
diff --git a/core/types/block.go b/core/types/block.go
@@ -129,6 +129,11 @@ func (h *Header) SanityCheck() error {
 	if eLen := len(h.Extra); eLen > 100*1024 {
 		return fmt.Errorf("too large block extradata: size %d", eLen)
 	}
+	if h.BaseFee != nil {
+		if bfLen := h.BaseFee.BitLen(); bfLen > 256 {
+			return fmt.Errorf("too large base fee: bitlen %d", bfLen)
+		}
+	}
 	return nil
 }
 

Original file line number	Diff line number	Diff line change
`@@ -129,6 +129,11 @@ func (h *Header) SanityCheck() error {`
`129`	`129`	`if eLen := len(h.Extra); eLen > 100*1024 {`
`130`	`130`	`return fmt.Errorf("too large block extradata: size %d", eLen)`
`131`	`131`	`}`
	`132`	`+ if h.BaseFee != nil {`
	`133`	`+ if bfLen := h.BaseFee.BitLen(); bfLen > 256 {`
	`134`	`+ return fmt.Errorf("too large base fee: bitlen %d", bfLen)`
	`135`	`+ }`
	`136`	`+ }`
`132`	`137`	`return nil`
`133`	`138`	`}`
`134`	`139`