tree de86aeb02160e57a598eaef4f76dcd74a26808e8
parent a968c2755e68ba71f7ddd221420a7d9dacba7bd1
author Henry Schreiner <HenrySchreinerIII@gmail.com> 1791262968 -0400
committer GitHub <noreply@github.com> 1791262968 -0700
gpgsig -----BEGIN PGP SIGNATURE-----
 
 wsFcBAABCAAQBQJqxID5CRC1aQ7uu5UhlAAA8iMQAJYCGOQES3MWHUelEdt6zbEK
 IN9oaciEwEkqYCxSUAEsBgFKZDPRSbazFbzl+0RjTtgaYPtdEyjhwixC8GOEkmWm
 xr0ufJbwqt8tJ6/GZagHCkwu72rs5sGaix90A9Qu55PwD/wQBqZb2uw+VnaUEySg
 dHG5DouRW2qrd9q1jXqIaydcMifmSE3qWucsnXrCIG4/VAsqKwcy4lhfkhhsnEB/
 IHI7RUUYIQBidMNs6e7M7uuvQgRRc74XP4C1XfVVP1iLW1G5afAeXSjagOUSoH5Y
 j0vDRtmtvuusFoNCsnvgiCobi44vXAnOmtOlOU6fK/0IS1UExcJvxJkwPjbn7po7
 /NIlfSpJcj0wICk38myR6L0GPvufXjFlM/fNd3bSS7hXFbJWxyjM7nH+fiXJFjMh
 /66g5MY32i0e0+kX9A/R2RvjNGFDPU7p8OiVZEsgAR67/goqjzyERXWhppqF8VSj
 HXSJA7oDlz4GmPtuU8PrNYc5MooK3amv1Rk53NqHtkZROv2Dmy0kfFolpPYgWzkI
 fBSGyIRuoNhiDIepFe/c2HTW8T6KtNOCwQZhPdx8pa+EIZ+KdAvi+aeMjrPuI19x
 V/Ce+5cgPtoNhymWkpfc6k6PAlkUMHXg2J+rv5N0KNGCUj9ogUbL3z8VUL2NhTj7
 +2KymfO3tcvpiHb1w8B/
 =Fwg+
 -----END PGP SIGNATURE-----
 

fix: only cache the internals pp after a successful lookup (#6190)

* fix: only cache the internals pp after a successful lookup

In get_pp(), last_istate_tls was set before get_or_create_pp_in_state_dict().
If that call threw, the thread cache matched the interpreter but held a null
pp, so later calls returned nullptr ("get_internals: get_pp() returned
nullptr"). Seen intermittently on MinGW in
test_import_in_subinterpreter_concurrently.

Assisted-by: ClaudeCode:claude-opus-5-5

* test: cover internals cache retries after failed lookup

---------

Co-authored-by: Ralf W. Grosse-Kunstleve <rgrossekunst@nvidia.com>