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

Insufficient VM stack overflow checking #38

Open
darrylabbate opened this issue Nov 18, 2022 · 0 comments
Open

Insufficient VM stack overflow checking #38

darrylabbate opened this issue Nov 18, 2022 · 0 comments
Labels
bug Something isn't working implementation General implementation of the interpreter runtime Virtual machine/runtime
Milestone
0.6.0

Comments

@darrylabbate
Copy link
Member

At the moment, there is only a single check for stack overflow at the very start of the interpreter loop. This obviously doesn't account for values being pushed onto the stack during the loop itself.
@darrylabbate darrylabbate added bug Something isn't working implementation General implementation of the interpreter runtime Virtual machine/runtime labels Nov 18, 2022
@darrylabbate darrylabbate modified the milestones: Riff 0.4, Riff 0.5 Nov 18, 2022
@darrylabbate darrylabbate changed the title VM stack overflows are not comprehensively checked Insufficient VM stack overflow checking Oct 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working implementation General implementation of the interpreter runtime Virtual machine/runtime
Projects
None yet
Milestone
0.6.0
Development

No branches or pull requests
1 participant
@darrylabbate