TrueCloudLab/neoneo-go
7
0
Fork 2
Code Issues Pull requests Projects Releases Packages Wiki Activity

vm: update stack size in SETITEM properly

Browse source
This commit is contained in:
Evgenii Stratonikov 2020-05-21 14:57:16 +03:00
parent d0853c0471
commit cd8445aa59
2 changed files with 22 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
pkg/vm/vm.go
View file

@ -1051,8 +1051,8 @@ func (v *VM) execute(ctx *Context, op opcode.Opcode, parameter []byte) (err erro
arr[index] = item arr[index] = item
v.estack.updateSizeAdd(arr[index]) v.estack.updateSizeAdd(arr[index])
case *MapItem: case *MapItem:
if t.Has(key.value) { if i := t.Index(key.value); i >= 0 {
v.estack.updateSizeRemove(item) v.estack.updateSizeRemove(t.value[i].Value)
} else if len(t.value) >= MaxArraySize { } else if len(t.value) >= MaxArraySize {
panic("too big map") panic("too big map")
} }

20
pkg/vm/vm_test.go
View file

@ -1442,6 +1442,26 @@ func TestSETITEMBigMapBad(t *testing.T) {
checkVMFailed(t, vm) checkVMFailed(t, vm)
} }
// This test checks is SETITEM properly updates reference counter.
// 1. Create 2 arrays of size MaxArraySize - 3. (MaxStackSize = 2 * MaxArraySize)
// 2. SETITEM each of them to a map.
// 3. Replace each of them with a scalar value.
func TestSETITEMMapStackLimit(t *testing.T) {
size := MaxArraySize - 3
m := NewMapItem()
m.Add(NewBigIntegerItem(1), NewArrayItem(makeArrayOfFalses(size)))
m.Add(NewBigIntegerItem(2), NewArrayItem(makeArrayOfFalses(size)))
prog := makeProgram(
opcode.DUP, opcode.PUSH1, opcode.PUSH1, opcode.SETITEM,
opcode.DUP, opcode.PUSH2, opcode.PUSH2, opcode.SETITEM,
opcode.DUP, opcode.PUSH3, opcode.PUSH3, opcode.SETITEM,
opcode.DUP, opcode.PUSH4, opcode.PUSH4, opcode.SETITEM)
v := load(prog)
v.estack.PushVal(m)
runVM(t, v)
}
func TestSETITEMBigMapGood(t *testing.T) { func TestSETITEMBigMapGood(t *testing.T) {
prog := makeProgram(opcode.SETITEM) prog := makeProgram(opcode.SETITEM)
vm := load(prog) vm := load(prog)