RFR: 8346881: [ubsan] logSelection.cpp:154:24 / logSelectionList.cpp:72:94 : runtime error: applying non-zero offset 1 to null pointer [v2]

[Matthias Baesken]

> When running jtreg tests on macOS aarch64 with ubsanized binaries, the following error is reported :
> 
> 
> jdk/src/hotspot/share/logging/logSelection.cpp:154:24: runtime error: applying non-zero offset 1 to null pointer
> UndefinedBehaviorSanitizer:DEADLYSIGNAL
> UndefinedBehaviorSanitizer: nested bug in the same thread, aborting.
> 
> 
> (XCode 13.1 was used)
> 
> This can be seen e.g. in these jdk jtreg tests :
> jdk/internal/misc/CDS/ArchivedEnumTest.java
> jdk/jfr/event/gc/collection/TestGCGarbageCollectionEvent.java
> jdk/jfr/startupargs/TestDumpOnExit.java
> 
> There is another issue observed here (also when running on macOS aarch64) , seems this happens in the for loop when incrementing comma_pos
> " for (char *comma_pos = copy, *cur = copy; success && comma_pos != nullptr; cur = comma_pos + 1) { ..."
> 
> 
> src/hotspot/share/logging/logSelectionList.cpp:72:94: runtime error: applying non-zero offset 1 to null pointer
> UndefinedBehaviorSanitizer:DEADLYSIGNAL
> UndefinedBehaviorSanitizer: nested bug in the same thread, aborting.

Matthias Baesken has updated the pull request incrementally with one additional commit since the last revision:

  for loop check has been moved

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/22903/files
  - new: https://git.openjdk.org/jdk/pull/22903/files/d25d806b..9fd0a5a3

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

  Stats: 1 line in 1 file changed: 0 ins; 0 del; 1 mod
  Patch: https://git.openjdk.org/jdk/pull/22903.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/22903/head:pull/22903

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