Fix GetPayloadTimelinessCommittee to return correct PTC size (#14012)

prysmaticlabs · May 24, 2024 · d110677 · d110677
1 parent 85d8b1a
commit d110677
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/beacon-chain/core/helpers/payload_attestation.go b/beacon-chain/core/helpers/payload_attestation.go
@@ -76,7 +76,7 @@ func GetPayloadTimelinessCommittee(ctx context.Context, state state.ReadOnlyBeac
 	}
 	committeesPerSlot := math.LargestPowerOfTwo(math.Min(SlotCommitteeCount(activeCount), fieldparams.PTCSize))
 	membersPerCommittee := fieldparams.PTCSize / committeesPerSlot
-	for i := uint64(0); i <= committeesPerSlot; i++ {
+	for i := uint64(0); i < committeesPerSlot; i++ {
 		committee, err := BeaconCommitteeFromState(ctx, state, slot, primitives.CommitteeIndex(i))
 		if err != nil {
 			return nil, err

diff --git a/beacon-chain/core/helpers/payload_attestation_test.go b/beacon-chain/core/helpers/payload_attestation_test.go
@@ -84,6 +84,7 @@ func TestGetPayloadTimelinessCommittee(t *testing.T) {
 
 	ptc, err := helpers.GetPayloadTimelinessCommittee(ctx, state, state.Slot())
 	require.NoError(t, err)
+	require.Equal(t, fieldparams.PTCSize, len(ptc))
 
 	committee1, err := helpers.BeaconCommitteeFromState(ctx, state, state.Slot(), 0)
 	require.NoError(t, err)