[jdk11u-dev] Integrated: 8276125: RunThese24H.java SIGSEGV in JfrThreadGroup::thread_group_id
Jiawei Tang
duke at openjdk.org
Mon Mar 4 14:50:59 UTC 2024
On Tue, 30 Jan 2024 09:30:06 GMT, Jiawei Tang <duke at openjdk.org> wrote:
> 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
This pull request has now been integrated.
Changeset: a4a5c7fe
Author: Jiawei Tang <tjw378335 at alibaba-inc.com>
Committer: Jie Fu <jiefu at openjdk.org>
URL: https://git.openjdk.org/jdk11u-dev/commit/a4a5c7fe66679ce4d9022443e3cd1f9ec38059ac
Stats: 5 lines in 1 file changed: 4 ins; 0 del; 1 mod
8276125: RunThese24H.java SIGSEGV in JfrThreadGroup::thread_group_id
Reviewed-by: mgronlun
Backport-of: a885aab696777c99c8c4c5d9a46afc5fe0a4fe47
-------------
PR: https://git.openjdk.org/jdk11u-dev/pull/2501
More information about the jdk-updates-dev
mailing list