Skip to content

feat(parser): add recursion depth limit to prevent stack overflow - #489

Open
AdamMagued wants to merge 1 commit into
alecthomas:masterfrom
AdamMagued:fix-issue-480
Open

AdamMagued wants to merge 1 commit into
alecthomas:masterfrom
AdamMagued:fix-issue-480

Conversation

@AdamMagued

Copy link
Copy Markdown

Fixes #480

Summary

  • Added configurable recursion depth limit to protect recursive descent parser from stack overflow.
  • Added unit tests testing deep recursion protection.

Verified via container sandbox test suite.

Signed-off-by: AdamMagued <adamismailmageud@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Parser stack overflow on deeply nested input (no recursion depth limit)

1 participant