RFR: 8308764: Reporting errors from create_vm may crash [v5]

David Holmes dholmes at openjdk.org
Wed Jun 7 00:44:06 UTC 2023


On Tue, 6 Jun 2023 22:50:30 GMT, David Holmes <dholmes at openjdk.org> wrote:

>> Sorry but this is getting a bit out of hand. Rearranging the deckchairs really doesn't change anything. You could argue that everything should happen in create_vm so JNI_CreateJavaVM is just a minimal wrapper, but that is not what fixing this issue is about. All I'm trying to do here is avoid the crash when there is no longer a current thread.
>
> New version committed.

New version reverted. You can't just do the exception check here because there is other code below that can thrown exceptions.

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/14238#discussion_r1220605376


More information about the hotspot-runtime-dev mailing list