From fb18f09e5527d912b222461d1c15b639ae249ca7 Mon Sep 17 00:00:00 2001 From: Etan Kissling Date: Sat, 4 Nov 2023 10:25:06 +0100 Subject: [PATCH] align `Blobs` limit for SSZ transport (#5558) Capacity should be set to theoretical limit to ensure correct hash root. Actual length may be shorter. Only use is `ExecutionPayloadForSigning` so it doesn't matter yet in practice, but still worth fixing. --- beacon_chain/spec/datatypes/deneb.nim | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/beacon_chain/spec/datatypes/deneb.nim b/beacon_chain/spec/datatypes/deneb.nim index 250314df48..f58a8accec 100644 --- a/beacon_chain/spec/datatypes/deneb.nim +++ b/beacon_chain/spec/datatypes/deneb.nim @@ -38,7 +38,7 @@ const type KzgCommitments* = List[KzgCommitment, Limit MAX_BLOB_COMMITMENTS_PER_BLOCK] - Blobs* = List[Blob, Limit MAX_BLOBS_PER_BLOCK] + Blobs* = List[Blob, Limit MAX_BLOB_COMMITMENTS_PER_BLOCK] # TODO this apparently is suppposed to be SSZ-equivalent to Bytes32, but # current spec doesn't ever SSZ-serialize it or hash_tree_root it