Skip to content

Stdlib: openssl_pkey_get_details AOT happy-path via libcrypto leaf (#34030) - #34036

Closed
PurHur wants to merge 1 commit into
masterfrom
fix/34030-openssl-pkey-get-details-aot
Closed

PurHur wants to merge 1 commit into
masterfrom
fix/34030-openssl-pkey-get-details-aot

Conversation

@PurHur

@PurHur PurHur commented Aug 23, 2026

Copy link
Copy Markdown
Owner

Summary

Test plan

  • ./script/aot-smoke.sh → 8/8
  • ./script/phpunit.sh --filter 'Issue34030OpensslPkeyGetDetailsHappyPathAotTest|OpensslPkeyGetDetailsAotTest' → OK (7 tests, 36 assertions)
  • Repro: PHP_COMPILER_HELPER_RUNTIME_O=0 php bin/compile.php -o /tmp/gd.bin test/repro/issue_34030_openssl_pkey_get_details_aot.php && /tmp/gd.bin → 512 / 0 / has-key
  • php script/check-selfhost-spine-coverage-sync.php → OK
  • Capability matrix regenerated (openssl_pkey_get_details VM/JIT/AOT yes)

Made with Cursor

…34030)

Close the #33496 leftover: thin AOT loads __osslPem from OpenSSLAsymmetricKey
and builds bits/type/key through an EVP leaf (peer openssl_pkey_new #34015),
with NestedJIT + compile-time bake for non-thin / known PEM.

Co-authored-by: Cursor <cursoragent@cursor.com>
@PurHur

PurHur commented Aug 23, 2026

Copy link
Copy Markdown
Owner Author

Superseded by #34035 (already on master @ 1c7dfaf) which closed #34030. This branch conflicts and duplicates that landing — closing without merge.

@PurHur

PurHur commented Aug 23, 2026

Copy link
Copy Markdown
Owner Author

Superseded by #34035 which already merged the same #34030 fix while this branch was in flight. Closing as duplicate.

@PurHur PurHur closed this Aug 23, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Stdlib: openssl_pkey_get_details() JIT/AOT happy-path leftover of #33496 — details array AOT (ext/openssl/openssl.c)

1 participant