Skip to content

Commit 9bab01b

Browse files
authored
consensus/clique: fix blob gas error message formatting (#33296)
Fixes error messages to print the actual blob gas value instead of the pointer address by dereferencing `ExcessBlobGas`, `BlobGasUsed` and `ParentBeaconRoot`.
1 parent 6452b7a commit 9bab01b

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

consensus/clique/clique.go

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -305,11 +305,11 @@ func (c *Clique) verifyHeader(chain consensus.ChainHeaderReader, header *types.H
305305
// Verify the non-existence of cancun-specific header fields
306306
switch {
307307
case header.ExcessBlobGas != nil:
308-
return fmt.Errorf("invalid excessBlobGas: have %d, expected nil", header.ExcessBlobGas)
308+
return fmt.Errorf("invalid excessBlobGas: have %d, expected nil", *header.ExcessBlobGas)
309309
case header.BlobGasUsed != nil:
310-
return fmt.Errorf("invalid blobGasUsed: have %d, expected nil", header.BlobGasUsed)
310+
return fmt.Errorf("invalid blobGasUsed: have %d, expected nil", *header.BlobGasUsed)
311311
case header.ParentBeaconRoot != nil:
312-
return fmt.Errorf("invalid parentBeaconRoot, have %#x, expected nil", header.ParentBeaconRoot)
312+
return fmt.Errorf("invalid parentBeaconRoot, have %#x, expected nil", *header.ParentBeaconRoot)
313313
}
314314
// All basic checks passed, verify cascading fields
315315
return c.verifyCascadingFields(chain, header, parents)

0 commit comments

Comments
 (0)