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

[Vladimir Kozlov]

On Thu, 29 May 2025 13:36:40 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 three additional commits since the last revision:
> 
>  - Merge branch 'master' into JDK-8357047
>  - 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>

Thank you, I submitted new testing.

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

PR Comment: https://git.openjdk.org/jdk/pull/25378#issuecomment-2920348137