Merge pull request #2 from tuyennhv/tuyennhv/rewards-api

fix: clone states to compute block rewards
ChainSafe · Feb 20, 2024 · eceea02 · eceea02
2 parents 719ab1d + 51bbf5e
commit eceea02
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/packages/beacon-node/src/chain/chain.ts b/packages/beacon-node/src/chain/chain.ts
@@ -1001,6 +1001,6 @@ export class BeaconChain implements IBeaconChain {
       throw Error(`Pre-state is unavailable given block's parent root ${toHexString(block.parentRoot)}`);
     }
 
-    return computeBlockRewards(block, preState, postState);
+    return computeBlockRewards(block, preState.clone(), postState?.clone());
   }
 }
diff --git a/packages/beacon-node/src/chain/rewards/blockRewards.ts b/packages/beacon-node/src/chain/rewards/blockRewards.ts
@@ -94,6 +94,7 @@ function computeSyncAggregateReward(block: altair.BeaconBlock, preState: CachedB
     return 0; // phase0 block does not have syncAggregate
   }
 }
+
 /**
  * Calculate rewards received by block proposer for including proposer slashings.
  * All proposer slashing rewards go to block proposer and none to whistleblower as of Deneb