Skip to content

Ensure the position information in AST nodes created by the parser is always consistent #93351

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.

Sign up for GitHub

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

Jump to bottom
Closed
pablogsal opened this issue May 30, 2022 · 2 comments · Fixed by #93352
Closed

Ensure the position information in AST nodes created by the parser is always consistent #93351

pablogsal opened this issue May 30, 2022 · 2 comments · Fixed by #93352
Labels
3.10 only security fixes 3.11 only security fixes 3.12 only security fixes interpreter-core (Objects, Python, Grammar, and Parser dirs)

Comments

@pablogsal
Copy link
Member

If we create an AST node with invalid position information, the error is reported in the compiler when it validates the line numbers and end line numbers, which is too far away and debugging this can be quite challenging. To improve the debugging experience, ensure that the parser cannot create an AST node with invalid position information.
@pablogsal pablogsal mentioned this issue May 30, 2022
Merged
@AA-Turner AA-Turner added interpreter-core (Objects, Python, Grammar, and Parser dirs) 3.11 only security fixes 3.10 only security fixes 3.12 only security fixes labels May 30, 2022
pablogsal added a commit to pablogsal/cpython that referenced this issue May 30, 2022
@pablogsal
pythongh-93351: Validate the position information when compiling AST …
dde7b64 
…nodes
pablogsal added a commit to pablogsal/cpython that referenced this issue May 30, 2022
@pablogsal
fixup! pythongh-93351: Validate the position information when compili…
8d865bf 
…ng AST nodes
pablogsal added a commit that referenced this issue May 30, 2022
@pablogsal
gh-93351: Ensure the position information in AST nodes created by the…
5893b5d 
… parser is always consistent (GH-93352)
miss-islington pushed a commit to miss-islington/cpython that referenced this issue May 30, 2022
@pablogsal @miss-islington
pythongh-93351: Ensure the position information in AST nodes created …
9311b89 
…by the parser is always consistent (pythonGH-93352)

(cherry picked from commit 5893b5d)

Co-authored-by: Pablo Galindo Salgado <[email protected]>
miss-islington added a commit that referenced this issue May 30, 2022
@miss-islington @pablogsal
gh-93351: Ensure the position information in AST nodes created by the…
7f6e6ab 
… parser is always consistent (GH-93352)

(cherry picked from commit 5893b5d)

Co-authored-by: Pablo Galindo Salgado <[email protected]>
@hauntsaninja
Copy link
Contributor

This broke a project of mine https://github.com/hauntsaninja/pyp that does some AST tricks. Worth adding a news entry (possibly to Python 3.11 What's New as well)?

@pablogsal
Copy link
Member Author

I will add a news entry for this

@bedevere-bot bedevere-bot mentioned this issue Jul 23, 2022
Merged
@pablogsal pablogsal mentioned this issue Jul 23, 2022
Merged
pablogsal added a commit to pablogsal/cpython that referenced this issue Jul 23, 2022
@pablogsal
pythongh-93351: Add news entry and what's new entry for changes in py…
35c568d 
…thongh-93351
pablogsal added a commit that referenced this issue Jul 23, 2022
@pablogsal
gh-93351: Add news entry and what's new entry for changes in gh-93351 (…
9762572 
…#95175)
@bedevere-bot bedevere-bot mentioned this issue Jul 23, 2022
Merged
miss-islington pushed a commit to miss-islington/cpython that referenced this issue Jul 23, 2022
@pablogsal @miss-islington
pythongh-93351: Add news entry and what's new entry for changes in py…
0625b33 
…thongh-93351 (pythonGH-95175)

(cherry picked from commit 9762572)

Co-authored-by: Pablo Galindo Salgado <[email protected]>
pablogsal added a commit to pablogsal/cpython that referenced this issue Jul 23, 2022
@pablogsal
[3.11] pythongh-93351: Add news entry and what's new entry for change…
9361609 
…s in pythongh-93351 (pythonGH-95175)

(cherry picked from commit 9762572)

Co-authored-by: Pablo Galindo Salgado <[email protected]>
@bedevere-bot bedevere-bot mentioned this issue Jul 23, 2022
Merged
pablogsal added a commit that referenced this issue Jul 23, 2022
@pablogsal
[3.11] gh-93351: Add news entry and what's new entry for changes in g…
e8edbda 
…h-93351 (GH-95175) (#95178)

(cherry picked from commit 9762572)

Co-authored-by: Pablo Galindo Salgado <[email protected]>
ambv pushed a commit that referenced this issue Jul 26, 2022
@miss-islington @pablogsal
gh-93351: Add news entry and what's new entry for changes in gh-93351 (…
5e971e8 
…GH-95175) (GH-95177)

(cherry picked from commit 9762572)

Co-authored-by: Pablo Galindo Salgado <[email protected]>
@char101 char101 mentioned this issue Dec 17, 2022
Closed
@ayalash ayalash mentioned this issue Feb 2, 2023
Open
@Technologicat Technologicat mentioned this issue Sep 26, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
3.10 only security fixes 3.11 only security fixes 3.12 only security fixes interpreter-core (Objects, Python, Grammar, and Parser dirs)
Projects
None yet
Milestone
No milestone
3 participants
@AA-Turner @pablogsal @hauntsaninja