RFR(s): 8218543: ThreadsList handling during error reporting can crash

Daniel D. Daugherty daniel.daugherty at oracle.com
Fri Oct 11 14:44:38 UTC 2019

On 10/11/19 7:08 AM, Robbin Ehn wrote:
> Hi all, please review.
> Issue:
> https://bugs.openjdk.java.net/browse/JDK-8218543
> Changeset:
> http://cr.openjdk.java.net/~rehn/8218543/v1/webrev/index.html

     No comments (thanks for removing the friends).

     No comments.

     No comments.

     No comments.

Thumbs up!


> We avoid the assert and let the error reporting use the ThreadsList in 
> an unsafe way. More discussion in issue.
> Note that there was an get_first_error_tid(), but for some reason 
> direct access to that field with friend was used. I change that to use 
> the get method and renamed the field following private _ naming 
> convention.
> Passes t1-2
> Thanks Robbin

More information about the hotspot-runtime-dev mailing list