RFR: 8346866: [ASAN] memoryReserver.cpp reported applying non-zero offset to non-null pointer produced null pointer [v3]

[SendaoYan]

On Thu, 2 Jan 2025 11:27:29 GMT, Stefan Karlsson <stefank at openjdk.org> wrote:

> I'm also not sure if the `<=` in `if (p2u(attach_point) <= stepsize)` should be `<=` and not `<`.

It should be `<=`, just avoid at the last for loop `attach_point` produced null pointer.

> use offsets in the loop logic instead
I think the solution use offset in the loop is better than additional exit point. The PR has been apply your advice.

And I think `lowest_start` will never receive nullptr. If `lowest_start` receive nullptr, the saniziter will report 'applying non-zero offset to null pointer'

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

PR Comment: https://git.openjdk.org/jdk/pull/22897#issuecomment-2568635144