[issue41261] 3.9-dev SEGV in object_recursive_isinstance in ast.literal_eval

Change by STINNER Victor : -- pull_requests: +21940 pull_request: https://github.com/python/cpython/pull/23024 ___ Python tracker ___

[issue41261] 3.9-dev SEGV in object_recursive_isinstance in ast.literal_eval

STINNER Victor added the comment: This bug should be fixed by: New changeset d2bea2636d5f0c2b196966315790af8e79c7bf82 by Victor Stinner in branch '3.9': [3.9] bpo-41194: Convert _ast extension to PEP 489 (GH-21807)

[issue41261] 3.9-dev SEGV in object_recursive_isinstance in ast.literal_eval

STINNER Victor added the comment: > From what I understand an RC1 is being released for 3.9. This is a gentle > reminder that there is no fix for this yet. I wrote PR 21807 to backport my changes from master to 3.9. My PR fix a crash on 3.9 with the following script: --- import sys

[issue41261] 3.9-dev SEGV in object_recursive_isinstance in ast.literal_eval

Change by Karthikeyan Singaravelan : -- nosy: +lukasz.langa ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue41261] 3.9-dev SEGV in object_recursive_isinstance in ast.literal_eval

Arcadiy Ivanov added the comment: >From what I understand an RC1 is being released for 3.9. This is a gentle >reminder that there is no fix for this yet. -- ___ Python tracker

[issue41261] 3.9-dev SEGV in object_recursive_isinstance in ast.literal_eval

Arcadiy Ivanov added the comment: Nevermind, `ast`/`_ast` is not pre-loaded by default, so it's in fact getting GC'ed after the first `run_path` is executed. -- ___ Python tracker

[issue41261] 3.9-dev SEGV in object_recursive_isinstance in ast.literal_eval

Arcadiy Ivanov added the comment: > If sys.modules['_ast'] is cleared and then _ast is imported again, > _PyState_AddModule() is called to store the new _ast module instance which > calls astmodule_clear() on the old module instance. I'm confused about this. In the repro below the

[issue41261] 3.9-dev SEGV in object_recursive_isinstance in ast.literal_eval

Arcadiy Ivanov added the comment: Yes, indeed, please, because I don't know what I would do for 3.9 if this is not fixed. :D -- ___ Python tracker ___

[issue41261] 3.9-dev SEGV in object_recursive_isinstance in ast.literal_eval

Change by Batuhan Taskaya : -- nosy: +BTaskaya ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue41261] 3.9-dev SEGV in object_recursive_isinstance in ast.literal_eval

STINNER Victor added the comment: It would be nice to fix this 3.9 regression before 3.9 final. -- priority: normal -> release blocker ___ Python tracker ___

[issue41261] 3.9-dev SEGV in object_recursive_isinstance in ast.literal_eval

Change by STINNER Victor : -- nosy: +dino.viehland ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue41261] 3.9-dev SEGV in object_recursive_isinstance in ast.literal_eval

STINNER Victor added the comment: This bug is a follow-up of bpo-41194: "Python 3.9.0b3 crash on compile() in PyAST_Check() when the _ast module is loaded more than once". -- ___ Python tracker

[issue41261] 3.9-dev SEGV in object_recursive_isinstance in ast.literal_eval

STINNER Victor added the comment: Ok, I can reproduce the crash on the 3.9 branch using: ./python repro.py. The first problem is that astmodule_clear() doesn't reset the initiallized member: add "state->initialized = 0;". The _ast module is special. Not only it has regular module function

[issue41261] 3.9-dev SEGV in object_recursive_isinstance in ast.literal_eval

Change by Arcadiy Ivanov : -- nosy: +vstinner ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue41261] 3.9-dev SEGV in object_recursive_isinstance in ast.literal_eval

Arcadiy Ivanov added the comment: $ PYTHONWARNINGS=ignore gdb --args /home/arcivanov/Documents/src/arcivanov/pybuilder/target/venv/test/cpython-3.9.0.beta.4/bin/python ./repro.py GNU gdb (GDB) Fedora 9.1-5.fc32 Copyright (C) 2020 Free Software Foundation, Inc. License GPLv3+: GNU GPL

[issue41261] 3.9-dev SEGV in object_recursive_isinstance in ast.literal_eval

Arcadiy Ivanov added the comment: This is in the 3.9 branch as of a0a6f1167834c87f12e2eca11dd77143103e7691 -- ___ Python tracker ___

[issue41261] 3.9-dev SEGV in object_recursive_isinstance in ast.literal_eval

New submission from Arcadiy Ivanov : "Short" reproducer: repro.py: ``` import sys from os import getcwd, chdir from runpy import run_path def smoke_test(script, *args): old_argv = list(sys.argv) del sys.argv[:] sys.argv.append(script) sys.argv.extend(args) old_modules