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

Engine.Execute terminates the process by throwing a StackOverflowException (even with recursion limit) #572

Closed
Metalnem opened this issue Dec 24, 2018 · 2 comments

Comments

@Metalnem
Copy link

@Metalnem Metalnem commented Dec 24, 2018

Engine.Execute terminates the process by throwing a StackOverflowException when executing script from the attached archive (even with recursion limit). You can run the following code to reproduce it (the path variable should contain the path to the extracted file):

var text = File.ReadAllText(path);
new Engine(options => options.LimitRecursion(1)).Execute(text);

Found via SharpFuzz.

@ayende
Copy link
Contributor

@ayende ayende commented Dec 24, 2018

Very nice find!

@lahma
Copy link
Collaborator

@lahma lahma commented May 28, 2019

Thank you for reporting this. This is a problem on Esprima's side where a lot of recursion is happening in this case, I've filed a separate issue in Esprima's repo: sebastienros/esprima-dotnet#104 .

@lahma lahma closed this May 28, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
3 participants