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

Ashutosh Mehra asmehra at openjdk.org
Thu May 22 13:25:36 UTC 2025


> 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>

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/25378/files
  - new: https://git.openjdk.org/jdk/pull/25378/files/649b08f0..d4c9e95b

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=25378&range=01
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=25378&range=00-01

  Stats: 7547 lines in 173 files changed: 6641 ins; 300 del; 606 mod
  Patch: https://git.openjdk.org/jdk/pull/25378.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/25378/head:pull/25378

PR: https://git.openjdk.org/jdk/pull/25378


More information about the hotspot-dev mailing list