Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Stack increase should emit StackCollision #19

Closed
vlopes11 opened this issue Aug 23, 2021 · 0 comments
Closed

Stack increase should emit StackCollision #19

vlopes11 opened this issue Aug 23, 2021 · 0 comments
Labels
fuel-vm Related to the `fuel-vm` crate.

Comments

@vlopes11
Copy link
Contributor

Nit: shouldn't this be called something like collision instead of overflow?

Originally posted by @adlerjohn in #17 (comment)
@vlopes11 vlopes11 mentioned this issue Aug 23, 2021
Merged
vlopes11 added a commit that referenced this issue Aug 23, 2021
@vlopes11
Add StackHeapCollision error
25276dd 
If an attempt to increase the stack space results in memory intersection
with the heap, then `ExecuteError::StackHeapCollision` should be
returned

Resolves #19
@vlopes11 vlopes11 mentioned this issue Aug 23, 2021
Closed
@adlerjohn adlerjohn linked a pull request Aug 26, 2021 that will close this issue
Refactor interpreter flow module #14
Closed
ControlCplusControlV pushed a commit that referenced this issue Nov 24, 2022
@vlopes11
Add RETD opcode (#19)
a30a54d
ControlCplusControlV pushed a commit that referenced this issue Nov 24, 2022
@vlopes11
Bump to v0.2.0 (#19)
f6e47d9
ControlCplusControlV pushed a commit that referenced this issue Dec 1, 2022
@vlopes11
Add RETD opcode (#19)
157beb4
ControlCplusControlV pushed a commit that referenced this issue Dec 1, 2022
@vlopes11
Bump to v0.2.0 (#19)
f73e297
mitchmindtree pushed a commit that referenced this issue Dec 5, 2022
@vlopes11
Bump to 0.4.2 (#19)
bf28048
@mitchmindtree mitchmindtree added the fuel-vm Related to the `fuel-vm` crate. label Dec 9, 2022
xgreenx pushed a commit that referenced this issue Dec 20, 2022
@vlopes11 @xgreenx
Bump to v0.2.0 (#19)
830d0f2
xgreenx pushed a commit that referenced this issue Dec 20, 2022
@vlopes11 @xgreenx
Add RETD opcode (#19)
c442a67
xgreenx pushed a commit that referenced this issue Dec 20, 2022
@vlopes11 @xgreenx
Bump to 0.4.2 (#19)
6565632
xgreenx pushed a commit that referenced this issue Dec 20, 2022
@vlopes11 @xgreenx
Bump to v0.2.0 (#19)
3dbd8ab
xgreenx pushed a commit that referenced this issue Dec 20, 2022
@vlopes11 @xgreenx
Add RETD opcode (#19)
f623678
xgreenx pushed a commit that referenced this issue Dec 20, 2022
@vlopes11 @xgreenx
Bump to 0.4.2 (#19)
1dded69
@freesig freesig closed this as completed May 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
fuel-vm Related to the `fuel-vm` crate.
Projects
Fuel Network
Status: Done
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Add StackHeapCollision error FuelLabs/fuel-vm
Refactor interpreter flow module FuelLabs/fuel-vm
3 participants
@mitchmindtree @freesig @vlopes11