RFR: 8358289: [asan] runtime/cds/appcds/aotCode/AOTCodeFlags.java reports heap-buffer-overflow in ArchiveBuilder [v3]
Ioi Lam
iklam at openjdk.org
Tue Jun 3 23:35:21 UTC 2025
On Tue, 3 Jun 2025 18:17:29 GMT, Vladimir Kozlov <kvn at openjdk.org> wrote:
>> There is difference between AdapterFingerPrint allocation size [compute_size](https://github.com/openjdk/jdk/blob/master/src/hotspot/share/runtime/sharedRuntime.cpp#L2227) which may not be aligned to HeapWord size and [size](https://github.com/openjdk/jdk/blob/master/src/hotspot/share/runtime/sharedRuntime.cpp#L2421) used for copying during AOT cache build which is aligned and can be bigger than allocation size.
>>
>> I added asserts to `AdapterFingerPrint` and `AdapterHandlerEntry` to make sure sizes are correct. Both are used in AOT cache build.
>>
>> I also moved `FreeHeap()` from `~AdapterFingerPrint()` to enforce the comment and simplify executed code.
>>
>> Thanks to @MBaesken for finding the issue and @iklam for pointing the cause.
>>
>> Testing tier1-3, xcomp, stress. Higher tiers are still running.
>
> Vladimir Kozlov has updated the pull request incrementally with one additional commit since the last revision:
>
> Use one compute_size_in_words() method
Marked as reviewed by iklam (Reviewer).
-------------
PR Review: https://git.openjdk.org/jdk/pull/25604#pullrequestreview-2894383374
More information about the hotspot-dev
mailing list