Integrated: 8281214: Unsafe use of long in VMThread::setup_periodic_safepoint_if_needed

Afshin Zafari duke at openjdk.org
Mon Dec 5 13:33:14 UTC 2022


On Fri, 2 Dec 2022 09:08:22 GMT, Afshin Zafari <duke at openjdk.org> wrote:

> ### Description 
> In VMThread::setup_periodic_safepoint_if_needed there is an unsafe use of long:
> 
> long interval_ms = SafepointTracing::time_since_last_safepoint_ms();
> 
> because SafepointTracing::time_since_last_safepoint_ms returns jlong which can be bigger than long.
> 
> ### Patch
> Changed `long` to `jlong`.
> 
> ### Test
> local:  make test TEST=runtime/Thread
> mach5: tier1

This pull request has now been integrated.

Changeset: b9eec968
Author:    Afshin Zafari <afshin.zafari at oracle.com>
Committer: Robbin Ehn <rehn at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/b9eec96889abf7df6d2b479e294c7823fb3d5850
Stats:     1 line in 1 file changed: 0 ins; 0 del; 1 mod

8281214: Unsafe use of long in VMThread::setup_periodic_safepoint_if_needed

Reviewed-by: eastigeevich, shade

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

PR: https://git.openjdk.org/jdk/pull/11476


More information about the hotspot-runtime-dev mailing list