[jdk11u-dev] RFR: 8276125: RunThese24H.java SIGSEGV in JfrThreadGroup::thread_group_id [v3]
Jiawei Tang
duke at openjdk.org
Mon Mar 4 12:32:09 UTC 2024
> It isn't a clean backport for two reasons.
> 1. jdk11 cannot use nullptr, so it should use NULL instead.
> 2. `parent_thread_group_handle != NULL` will cause a compile error without a type cast.
>
> This backport prevents a JFR operation from accessing an illegal address which has not be filled before the initialization of a JavaThread, which is created from a JNI interface.
>
> The risk is very low since it only adds a null pointer check. Besides, This backport has be accepted in [jdk17-dev](https://github.com/openjdk/jdk17u-dev/pull/2179).
>
> Additional testing:
> - [x] Linux aarch64 server fastdebug, test/jdk/jdk/jfr passed
Jiawei Tang has updated the pull request incrementally with one additional commit since the last revision:
Update full name
-------------
Changes:
- all: https://git.openjdk.org/jdk11u-dev/pull/2501/files
- new: https://git.openjdk.org/jdk11u-dev/pull/2501/files/a929af31..e720c8a6
Webrevs:
- full: https://webrevs.openjdk.org/?repo=jdk11u-dev&pr=2501&range=02
- incr: https://webrevs.openjdk.org/?repo=jdk11u-dev&pr=2501&range=01-02
Stats: 0 lines in 0 files changed: 0 ins; 0 del; 0 mod
Patch: https://git.openjdk.org/jdk11u-dev/pull/2501.diff
Fetch: git fetch https://git.openjdk.org/jdk11u-dev.git pull/2501/head:pull/2501
PR: https://git.openjdk.org/jdk11u-dev/pull/2501
More information about the jdk-updates-dev
mailing list