Skip to content
This repository has been archived by the owner on Mar 20, 2024. It is now read-only.

[Bug] Implement runtime recursion limit #367

Open
brson opened this issue Sep 15, 2023 · 0 comments
Open

[Bug] Implement runtime recursion limit #367

brson opened this issue Sep 15, 2023 · 0 comments
Labels
bug Something isn't working

Comments

@brson
Copy link
Collaborator

brson commented Sep 15, 2023

Followup to #333

The move VM produces errors in some cases when asked to recurse deeply. move-native has no limits and will just crash if it hits the end of the stack.

The only case of this I have seen is in this test case in stdlib's bcs_tests:

    #[test]
    #[expected_failure] // VM_MAX_VALUE_DEPTH_REACHED
    fun encode_129() {
        bcs::to_bytes(&Box { x: box127(true) });
    }

But there are probably others.

@brson brson added the bug Something isn't working label Sep 15, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant