Fix getPayloadBodiesByRangeV1 fetching too many payloads (#3057)

bhaskar1001101 · web-flow · commit 577e3559498c · 2025-02-11T04:17:21.000Z
diff --git a/nimbus/beacon/api_handler/api_getbodies.nim b/nimbus/beacon/api_handler/api_getbodies.nim
@@ -71,7 +71,7 @@ proc getPayloadBodiesByRange*(ben: BeaconEngineRef,
     last = ben.chain.latestNumber
 
   # get bodies from database
-  for bn in start..ben.chain.baseNumber:
+  for bn in start..min(last, ben.chain.baseNumber):
     let blk = ben.chain.blockByNumber(bn).valueOr:
       result.add Opt.none(ExecutionPayloadBodyV1)
       continue
@@ -80,4 +80,5 @@ proc getPayloadBodiesByRange*(ben: BeaconEngineRef,
   if last > ben.chain.baseNumber:
     let blocks = ben.chain.blockFromBaseTo(last)
     for i in countdown(blocks.len-1, 0):
-      result.add Opt.some(toPayloadBody(blocks[i]))
+      if blocks[i].header.number >= start:
+        result.add Opt.some(toPayloadBody(blocks[i]))
