dstepanov-yadro/neoneo-go
1
0
Fork 0
forked from TrueCloudLab/neoneo-go
Code Pull requests Activity

core: pop all args from stack before validation checks in Notify

Browse source 
We should match the C# behaviour.
This commit is contained in:
Anna Shaleva 2021-03-31 13:54:53 +03:00
parent 793f27084b
commit 93530fa8fa
2 changed files with 3 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
pkg/core/interop/runtime/engine.go
View file

@ -52,11 +52,11 @@ func GetTrigger(ic *interop.Context) error {
// in neo-go the only meaningful thing to do here is to log.
func Notify(ic *interop.Context) error {
name := ic.VM.Estack().Pop().String()
elem := ic.VM.Estack().Pop()
args := elem.Array()
if len(name) > MaxEventNameLen {
return fmt.Errorf("event name must be less than %d", MaxEventNameLen)
}
elem := ic.VM.Estack().Pop()
args := elem.Array()
// But it has to be serializable, otherwise we either have some broken
// (recursive) structure inside or an interop item that can't be used
// outside of the interop subsystem anyway.

2
pkg/core/interop/runtime/engine_test.go
View file

@ -142,7 +142,7 @@ func TestNotify(t *testing.T) {
return ic
}
t.Run("big name", func(t *testing.T) {
ic := newIC(string(make([]byte, MaxEventNameLen+1)), []byte{42})
ic := newIC(string(make([]byte, MaxEventNameLen+1)), stackitem.NewArray([]stackitem.Item{stackitem.Null{}}))
require.Error(t, Notify(ic))
})
t.Run("recursive struct", func(t *testing.T) {