RFR: 8374341: Use of "\s" instead of "\\s" in CoreUtils.java results in splitting on backspace instead of whitespace [v4]
Kevin Walls
kevinw at openjdk.org
Fri Jan 9 12:18:38 UTC 2026
On Wed, 7 Jan 2026 12:48:29 GMT, Eunbin Son <duke at openjdk.org> wrote:
>> ### Summary
>> Fix incorrect regex usage in CoreUtils.getCoreFileLocation.
>>
>> ### Description
>> CoreUtils.getCoreFileLocation uses String.split("\s", 2).
>> The string literal "\s" represents a backspace character, not a whitespace regex.
>> As a result, the split operation does not separate fields on whitespace.
>> This change replaces the pattern with `"\\s+"` to correctly split on whitespace.
>> No other logic is modified.
>>
>> ### Bug ID : JDK-8374341
>> https://bugs.java.com/bugdatabase/view_bug?bug_id=8374341
>
> Eunbin Son has refreshed the contents of this pull request, and previous commits have been removed. The incremental views will show differences compared to the previous content of the PR. The pull request contains one new commit since the last revision:
>
> JDK-8374341: Remove unverified systemd-coredump handling code
Yes we should update, no need for a new JBS issue.
How about "Remove coredumpctl call in CoreUtils.getCoreFileLocation"
I can update the JBS issue if you cannot.
-------------
PR Comment: https://git.openjdk.org/jdk/pull/28984#issuecomment-3728671700
More information about the serviceability-dev
mailing list