Description

xiaxinmeng

opened on Apr 6, 2023

Issue body actions

ast.parse() takes a long string as the argument leading to a segfault.

test.py

import ast
x = ast.parse('+chr(33)'*1000000)

Crash message: Segmentation fault (core dumped)

Environment:
Ubuntu 18.04
rustpython v0.2.0

Activity

1. DimitrisJim commented on Apr 6, 2023

   

   DimitrisJim

   on Apr 6, 2023

   Member

   More actions

   Nice, this seems like a variation of #4862. Basically we're creating a deeply nested AST here since the rhs of every BinOp will be another BinOp so it fails while trying to recurse into this.

   Both these issues will most likely be solved by the same solution: checks while recursing deeply during the compilation phase (I'm guessing the AST traversal is the cause here but I'd need to check to make sure).

   Thanks a lot for reporting and feel free to re-open if you disagree.
2. DimitrisJim

   closed this as completedon Apr 6, 2023

Issue actions