forks/go-ethereum
1
0
Fork 1
mirror of https://github.com/ethereum/go-ethereum.git synced 2026-08-18 01:43:47 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 1

core/vm: nil check memorySize instead of dynamicGas

Browse source 
We have the assumption that if an opcode has memorySize function then it has
dynamicGas function. And we check that condition of every opcode in validate
function. But in the interpreter, we actually assume that if an opcode has
dynamicGas then it has memorySize function. However, it is still correct as
every opcode currently have either these 2 functions or none. So this commit is
trivial, to make the interpreter consistent with asssumption in validate
function to avoid confusion.
This commit is contained in:
Bui Quang Minh 2024-12-25 11:27:32 +07:00
parent 341647f186
commit 384ba784c5
No known key found for this signature in database
GPG key ID: DADFFFFF480828D6
1 changed files with 1 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
core/vm/interpreter.go
View file

@ -258,7 +258,7 @@ func (in *EVMInterpreter) Run(contract *Contract, input []byte, readOnly bool) (
contract.Gas -= cost
}
if operation.dynamicGas != nil {
if operation.memorySize != nil {
// All ops with a dynamic memory usage also has a dynamic gas cost.
var memorySize uint64
// calculate the new memory size and expand the memory to fit