RFR: 8357047: [ubsan] AdapterFingerPrint::AdapterFingerPrint runtime error: index 3 out of bounds [v2]

[Vladimir Kozlov]

On Thu, 22 May 2025 13:25:36 GMT, Ashutosh Mehra <asmehra at openjdk.org> wrote:

>> Refactor `AdapterFingerPrint` to remove the notion of compact representation. As the memory for storing fingerprint is always allocated next to the `AdapterFingerPrint` object, use of `_value` array is not required. This simplifies the implementation a bit, and would also fix the "false" positives flagged by ubsan when accessing `_value` array beyond its size (See the description in JBS issue).
>> 
>> Testing: hotspot_runtime on x86-64 with fastdebug build
>
> Ashutosh Mehra has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains two additional commits since the last revision:
> 
>  - Merge branch 'master' into JDK-8357047
>  - 8357047: [ubsan] AdapterFingerPrint::AdapterFingerPrint runtime error: index 3 out of bounds
>    
>    Signed-off-by: Ashutosh Mehra <asmehra at redhat.com>

Good. Let me test it.

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

PR Review: https://git.openjdk.org/jdk/pull/25378#pullrequestreview-2861598108