status-im
/
nimbus-eth1
mirror of https://github.com/status-im/nimbus-eth1.git
2
0
Fork 0
Code Issues Projects Releases Wiki Activity

Fix payload validation in engine_newPayload (#2107) 

* Fix validatePayload in engine_newPayload

* Fix copyright year
This commit is contained in:
andri lim 2024-03-26 22:13:30 +07:00 committed by GitHub
parent 33976e8875
commit ef378c88a5
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
1 changed files with 10 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
nimbus/beacon/api_handler/api_newpayload.nim
View File

@ -1,5 +1,5 @@
# Nimbus
# Copyright (c) 2023 Status Research & Development GmbH
# Copyright (c) 2023-2024 Status Research & Development GmbH
# Licensed under either of
# * Apache License, version 2.0, ([LICENSE-APACHE](LICENSE-APACHE))
# * MIT license ([LICENSE-MIT](LICENSE-MIT))
@ -48,6 +48,14 @@ template validateVersion(com, timestamp, version, apiVersion) =
" expect ExecutionPayload" & $apiVersion &
" but got ExecutionPayload" & $version)
template validatePayload(apiVersion, version, payload) =
if version == Version.V3:
if payload.blobGasUsed.isNone:
raise invalidParams("newPayload" & $apiVersion &
"blobGasUsed is expected from execution payload")
if payload.excessBlobGas.isNone:
raise invalidParams("newPayload" & $apiVersion &
"excessBlobGas is expected from execution payload")
proc newPayload*(ben: BeaconEngineRef,
apiVersion: Version,
@ -70,6 +78,7 @@ proc newPayload*(ben: BeaconEngineRef,
version = payload.version
validateVersion(com, timestamp, version, apiVersion)
validatePayload(apiVersion, version, payload)
var header = blockHeader(payload, ethHash beaconRoot)
let blockHash = ethHash payload.blockHash