RFR: 8346866: [ASAN] memoryReserver.cpp reported applying non-zero offset to non-null pointer produced null pointer

SendaoYan syan at openjdk.org
Tue Dec 31 04:08:13 UTC 2024


Hi all,
This PR add an extra loop condition check `p2u(attach_point) > stepsize` in function `HeapReserver::Instance::try_reserve_range` to make sure the loop will not wrap around, and make UndefinedBehaviorSanitizer silent. The change do not change the original logic, risk is low.

Additional testing:

- [ ]  jtreg tests(include tier1/2/3 etc., which include tests added by [PR22712](https://github.com/openjdk/jdk/pull/22712)) on linux-x64 with release build
- [ ]  jtreg tests(include tier1/2/3 etc., which include tests added by [PR22712](https://github.com/openjdk/jdk/pull/22712)) on linux-x64 with fastdebug build
- [ ]  jtreg tests(include tier1/2/3 etc., which include tests added by [PR22712](https://github.com/openjdk/jdk/pull/22712)) on linux-aarch64 with release build
- [ ]  jtreg tests(include tier1/2/3 etc., which include tests added by [PR22712](https://github.com/openjdk/jdk/pull/22712)) on linux-aarch64 with fastdebug build

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

Commit messages:
 - 8346866: [ASAN] memoryReserver.cpp reported applying non-zero offset to non-null pointer produced null pointer

Changes: https://git.openjdk.org/jdk/pull/22897/files
  Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=22897&range=00
  Issue: https://bugs.openjdk.org/browse/JDK-8346866
  Stats: 1 line in 1 file changed: 0 ins; 0 del; 1 mod
  Patch: https://git.openjdk.org/jdk/pull/22897.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/22897/head:pull/22897

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


More information about the hotspot-runtime-dev mailing list