RFR: 8374341: Use of "\s" instead of "\\s" in CoreUtils.java results in splitting on backspace instead of whitespace [v2]
Kevin Walls
kevinw at openjdk.org
Wed Jan 28 09:35:11 UTC 2026
On Tue, 27 Jan 2026 22:05:14 GMT, Stuart Marks <smarks at openjdk.org> wrote:
>> @thswlsqls Hi, are you able to edit the title of this PR to match the JBS issue?
>
> Note that the `\s` escape sequence denotes a SPACE character (U+0020) and not a BACKSPACE character as stated in the description. Thus the regex might actually match the relevant line in the example from @kevinjwalls' comment:
>
> # cat /proc/sys/kernel/core_pattern
> |/usr/lib/systemd/systemd-coredump %P %u %g %s %t %c %h %d %F
>
> if the character following `-coredump` is a space character.
>
> Of course this is moot if the decision is to remove this stuff entirely... but it appears that the original code might have worked, at least in some cases. That might resolve the "How could this ever have worked?!?" mystery.
Thanks @stuart-marks ! Yes of course. So the problem here is really coreadm not having permission to be useful when it's run, for most people. The split pattern was either an obscure choice at the time or a typo, but that part doesn't matter as much as this issue first suggested.
-------------
PR Comment: https://git.openjdk.org/jdk/pull/28984#issuecomment-3810083818
More information about the serviceability-dev
mailing list