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

[Matthias Baesken]

On Thu, 2 Jan 2025 14:12:41 GMT, Christoph Langer <clanger at openjdk.org> wrote:

>> Unfortunately we still run into the issue , even with the existing loop condition.
>> check the flowchart of the for loop 
>> https://www.programiz.com/cpp-programming/for-loop
>> the update condition is done after the body ; then we run into the issue.
>
> Ah, right, I see. But in that case, we can/should remove the check for comma_pos != nullptr from the condition because we're explicitly doing it in the loop body then.

Agree, the check can be simplified .

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

PR Review Comment: https://git.openjdk.org/jdk/pull/22903#discussion_r1900930274