[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