core: verify genesis extradata for clique (ethereum#24470)

gurukamath · fjl · holiman · sadoci · commit 981b7d3dffda · 2023-01-27T12:14:51.000+09:00
* Add extra-data checks for clique genesis

* Update genesis.go

* Update genesis.go

* core: simplify clique genesis check

Co-authored-by: Felix Lange &lt;fjl@twurst.com&gt;
Co-authored-by: Martin Holst Swende &lt;martin@swende.se&gt;
diff --git a/core/genesis.go b/core/genesis.go
@@ -404,7 +404,7 @@ func (g *Genesis) Commit(db ethdb.Database) (*types.Block, error) {
 	if err := config.CheckConfigForkOrder(); err != nil {
 		return nil, err
 	}
-	if config.Clique != nil && len(block.Extra()) == 0 {
+	if config.Clique != nil && len(block.Extra()) < 32+crypto.SignatureLength {
 		return nil, errors.New("can't start clique chain without signers")
 	}
 	if err := g.Alloc.write(db, block.Hash()); err != nil {

Original file line number	Diff line number	Diff line change
`@@ -404,7 +404,7 @@ func (g Genesis) Commit(db ethdb.Database) (types.Block, error) {`
`404`	`404`	`if err := config.CheckConfigForkOrder(); err != nil {`
`405`	`405`	`return nil, err`
`406`	`406`	`}`
`407`		`- if config.Clique != nil && len(block.Extra()) == 0 {`
	`407`	`+ if config.Clique != nil && len(block.Extra()) < 32+crypto.SignatureLength {`
`408`	`408`	`return nil, errors.New("can't start clique chain without signers")`
`409`	`409`	`}`
`410`	`410`	`if err := g.Alloc.write(db, block.Hash()); err != nil {`