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

Missing cyclic __proto__ value check #20

Open
y21 opened this issue Feb 19, 2022 · 0 comments
Open

Missing cyclic __proto__ value check #20

y21 opened this issue Feb 19, 2022 · 0 comments
Assignees
Labels
A-vm bug Something isn't working

Comments

@y21
Copy link
Owner

y21 commented Feb 19, 2022

let z = {};
let y = {__proto__:z};
z.__proto__ = y;
y.a

running this crashes dash with a stack overflow because it gets stuck in a recursive "loop" trying to look up property 'a' in its parent __proto__ object if it can't find it

Other JS engines have guards against that. V8 throws Cyclic __proto__ value

@y21 y21 added bug Something isn't working A-vm labels Feb 19, 2022
@y21 y21 self-assigned this Apr 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-vm bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant