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

Jump to bottom

Invalid child node access in ast.c #47627

Closed
kristjanvalur mannequin opened this issue Jul 16, 2008 · 2 comments
Closed

Invalid child node access in ast.c #47627

kristjanvalur mannequin opened this issue Jul 16, 2008 · 2 comments
Labels
interpreter-core (Objects, Python, Grammar, and Parser dirs) type-crash A hard crash of the interpreter, possibly with a core dump

Comments

@kristjanvalur
Copy link
Mannequin

kristjanvalur mannequin commented Jul 16, 2008

BPO 3377
Nosy @kristjanvalur

Note: these values reflect the state of the issue at the time it was migrated and might not reflect the current state.

Show more details

GitHub fields:

assignee = None
closed_at = <Date 2008-07-17.16:37:42.623>
created_at = <Date 2008-07-16.13:44:26.818>
labels = ['interpreter-core', 'type-crash']
title = 'Invalid child node access in ast.c'
updated_at = <Date 2008-07-17.16:37:42.622>
user = 'https://github.com/kristjanvalur'

bugs.python.org fields:

activity = <Date 2008-07-17.16:37:42.622>
actor = 'jhylton'
assignee = 'jhylton'
closed = True
closed_date = <Date 2008-07-17.16:37:42.623>
closer = 'jhylton'
components = ['Interpreter Core']
creation = <Date 2008-07-16.13:44:26.818>
creator = 'kristjan.jonsson'
dependencies = []
files = []
hgrepos = []
issue_num = 3377
keywords = []
message_count = 2.0
messages = ['69787', '69887']
nosy_count = 2.0
nosy_names = ['jhylton', 'kristjan.jonsson']
pr_nums = []
priority = 'normal'
resolution = None
stage = None
status = 'closed'
superseder = None
type = 'crash'
url = 'https://bugs.python.org/issue3377'
versions = ['Python 3.0']
@kristjanvalur
Copy link
Mannequin Author

kristjanvalur mannequin commented Jul 16, 2008

Purify complained about reading uninitialized memory in ast.c:752 of
two bytes which corresponds to the type field. Looking into this, line
750 increments i without checking that there are in fact this many
children.
If you add the line:
assert(i < NCH(n));
after line 750 you get an assertion failure when you run
test_keywordonlyarg in the testsuite

@kristjanvalur kristjanvalur mannequin added interpreter-core (Objects, Python, Grammar, and Parser dirs) type-crash A hard crash of the interpreter, possibly with a core dump labels Jul 16, 2008
@jhylton jhylton mannequin self-assigned this Jul 17, 2008
@jhylton
Copy link
Mannequin

jhylton mannequin commented Jul 17, 2008

Committed revision 65064.

@jhylton jhylton mannequin closed this as completed Jul 17, 2008
@ezio-melotti ezio-melotti transferred this issue from another repository Apr 10, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
interpreter-core (Objects, Python, Grammar, and Parser dirs) type-crash A hard crash of the interpreter, possibly with a core dump
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
0 participants