RFR: 8335217: Fix memory ordering in ClassLoaderData::ChunkedHandleList [v4]

Erik Österlund eosterlund at openjdk.org
Fri Jun 28 15:59:21 UTC 2024


On Fri, 28 Jun 2024 15:06:04 GMT, Stefan Karlsson <stefank at openjdk.org> wrote:

>> src/hotspot/share/classfile/classLoaderData.cpp line 184:
>> 
>>> 182: 
>>> 183: ClassLoaderData::ChunkedHandleList::~ChunkedHandleList() {
>>> 184:   Chunk* c = _head;
>> 
>> This really should not be needed and just becomes something that people will wonder about why it is there. If we ever have concurrent accesses here we have a use-after-free situation and have much bigger issues to deal with. I'd like to see this reverted.
>
> @zhengyu123 I still would like to see this line getting reverted.

I agree.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/19919#discussion_r1658954044


More information about the hotspot-runtime-dev mailing list