RFR: 8360791: [ubsan] Adjust signal handling [v2]
Magnus Ihse Bursie
ihse at openjdk.org
Mon Jun 30 08:59:41 UTC 2025
On Mon, 30 Jun 2025 07:02:31 GMT, Matthias Baesken <mbaesken at openjdk.org> wrote:
>> A couple of tests e.g. VendorInfoPluginsTest but also some Hotspot like runtime/ErrorHandling/CreateCoredumpOnCrash.java put (write) to special addresses like 0 to provoke crashs.
>> However this does not work well with ubsan-enabled binaries on the clang based platforms (macOS, AIX).
>> The mentioned tests generate a SIGSEGV.
>>
>> Some other tests generate a SIGBUS, e.g.
>>
>> runtime/memory/ReserveMemory.java
>> runtime/memory/ReadFromNoaccessArea.java
>>
>> and this leads to similar issues with ubsan-enabled binaries.
>>
>> We should adjust the signal handling with the sanitizer options, how to do this is documented here :
>> https://github.com/google/sanitizers/wiki/SanitizerCommonFlags
>
> Matthias Baesken has updated the pull request incrementally with one additional commit since the last revision:
>
> Remove the clang check
Looks good. Let's exclude the other signals if we can actually show it is a problem.
-------------
Marked as reviewed by ihse (Reviewer).
PR Review: https://git.openjdk.org/jdk/pull/26015#pullrequestreview-2970443722
More information about the build-dev
mailing list