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