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.

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

Why is limit of callstack 5? #309

Open
da1suk8 opened this issue Jun 16, 2023 · 0 comments
Open

Why is limit of callstack 5? #309

da1suk8 opened this issue Jun 16, 2023 · 0 comments
Labels
dynamic_link relate the dynamic link call feature

Comments

@da1suk8
Copy link
Member

da1suk8 commented Jun 16, 2023

Currently, the limit of the call stack is defined as 5.
It is mainly used in the following codes.

pub const DYNAMIC_CALL_DEPTH_LIMIT_CNT: usize = 5;

if ctx.dynamic_callstack.len() >= DYNAMIC_CALL_DEPTH_LIMIT_CNT {
return Err(VmError::dynamic_call_depth_over_limitation_err());
}

if callstack.len() >= DYNAMIC_CALL_DEPTH_LIMIT_CNT {
return Err(VmError::dynamic_call_depth_over_limitation_err());
};

Check callstack limits and fix if necessary.

@da1suk8 da1suk8 added the dynamic_link relate the dynamic link call feature label Jul 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dynamic_link relate the dynamic link call feature
Projects
None yet
Development

No branches or pull requests

1 participant