[issue42882] Restarting the interpreter causes UB on 3.10.0a4

STINNER Victor added the comment: New changeset 196d4deaf4810a0bba75ba537dd40f2d71a5a634 by Ken Jin in branch 'master': bpo-42882: Fix MSVC warnings in pystate.c (GH-24440) https://github.com/python/cpython/commit/196d4deaf4810a0bba75ba537dd40f2d71a5a634 --

[issue42882] Restarting the interpreter causes UB on 3.10.0a4

Change by Ken Jin : -- nosy: +kj nosy_count: 2.0 -> 3.0 pull_requests: +23250 pull_request: https://github.com/python/cpython/pull/24440 ___ Python tracker ___

[issue42882] Restarting the interpreter causes UB on 3.10.0a4

STINNER Victor added the comment: Ok, now I can safely close the issue ;-) -- resolution: -> fixed stage: patch review -> resolved status: open -> closed ___ Python tracker

[issue42882] Restarting the interpreter causes UB on 3.10.0a4

STINNER Victor added the comment: New changeset 11d13e83abedabba12b28773317f1a365113e7af by Victor Stinner in branch 'master': bpo-42882: Add test_embed.test_unicode_id_init() (GH-24198) https://github.com/python/cpython/commit/11d13e83abedabba12b28773317f1a365113e7af --

[issue42882] Restarting the interpreter causes UB on 3.10.0a4

STINNER Victor added the comment: Thanks for your bug report, and thanks for testing alpha versions of Python! It's now fixed. But I didn't feel comfortable without a regression test. So I wrote PR 24198 to add an unit test on _PyUnicode_FromId() with multiple Python initializations.

[issue42882] Restarting the interpreter causes UB on 3.10.0a4

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

[issue42882] Restarting the interpreter causes UB on 3.10.0a4

STINNER Victor added the comment: New changeset 44bf57aca627bd11a08b12fe4e4b6a0e1d268862 by Victor Stinner in branch 'master': bpo-42882: _PyRuntimeState_Init() leaves unicode next_index unchanged (GH-24193) https://github.com/python/cpython/commit/44bf57aca627bd11a08b12fe4e4b6a0e1d268862

[issue42882] Restarting the interpreter causes UB on 3.10.0a4

Yannick Jadoul added the comment: Wow, that was fast! Thanks! I tried this out locally, and all pybind11's tests pass now. We can try again once there's a nightly build or new alpha :-) -- ___ Python tracker

[issue42882] Restarting the interpreter causes UB on 3.10.0a4

STINNER Victor added the comment: Oh. In _PyUnicode_FromId(), I made the assumption that _PyRuntime is left unchanged when Py_Initialize()Py_Finalize() is called multiple times. But I was wrong, it is always reset to zero. So I wrote PR 24193 to explicitly save/restore

[issue42882] Restarting the interpreter causes UB on 3.10.0a4

Change by STINNER Victor : -- keywords: +patch pull_requests: +23020 stage: -> patch review pull_request: https://github.com/python/cpython/pull/24193 ___ Python tracker ___

[issue42882] Restarting the interpreter causes UB on 3.10.0a4

Change by STINNER Victor : -- nosy: +vstinner ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue42882] Restarting the interpreter causes UB on 3.10.0a4

New submission from Yannick Jadoul : Issue detected in the embedding tests of pybind11, running on the latest alpha of 3.10: https://github.com/pybind/pybind11/issues/2774 I have reduced the weird issue/crash to a minimal reproducer, which consistently reproduces the crash on my Linux