Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
add: Runtime.LoadScript #766
add: Runtime.LoadScript #766
Changes from all commits
1fedde3
2c23ba8
4e3296c
e25690f
727d6da
a422a17
768ed9f
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@erikzhang the unique weird thing is that, the current instruction was null after the dynamic script
op:[019D] INITSLOT op:[01A0] PUSHDATA1 op:[01A3] CONVERT op:[01A5] CONVERT op:[01A7] STLOC0 op:[01A8] LDARG1 op:[01A9] LDARG0 op:[01AA] PUSH2 op:[01AB] PACK op:[01AC] PUSH15 op:[01AD] LDLOC0 op:[01AE] SYSCALL op:[0000] ADD + op:[0001] op: [01B3] JMP op:[01B5] RET
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
After execute "ADD", the current instruction point will auto jump to "Next" point even it is an outrange point. (see here)
Dynamic Script length is 1, InstructionPointer +1, InstructionPointer will be 1.
In the next iteration, the out-range point will get a "null" instruction.
By the way, the vm will auto change "null" to "RET" instruction, so it won't be fault in vm(https://github.com/neo-project/neo-vm/blob/f48de7e595ade7c72be6ccd5cce0ab99708f0afd/src/Neo.VM/ExecutionEngine.cs#L1450).