Merge pull request #257 from OffchainLabs/fix-deserialize

Fix DeserializeHeaderExtraInformation that panics for nil header
OffchainLabs · Sep 26, 2023 · 07732c1 · 07732c1
2 parents e9028a6 + 45efc82
commit 07732c1
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/core/types/arb_types.go b/core/types/arb_types.go
@@ -504,7 +504,7 @@ func (info HeaderInfo) UpdateHeaderWithInfo(header *Header) {
 }
 
 func DeserializeHeaderExtraInformation(header *Header) HeaderInfo {
-	if header.BaseFee == nil || header.BaseFee.Sign() == 0 || len(header.Extra) != 32 || header.Difficulty.Cmp(common.Big1) != 0 {
+	if header == nil || header.BaseFee == nil || header.BaseFee.Sign() == 0 || len(header.Extra) != 32 || header.Difficulty.Cmp(common.Big1) != 0 {
 		// imported blocks have no base fee
 		// The genesis block doesn't have an ArbOS encoded extra field
 		return HeaderInfo{}