Integrated: 8341015: OopStorage location decoder crashes accessing non-initalized OopStorage

[Aleksey Shipilev]

On Thu, 26 Sep 2024 11:36:26 GMT, Aleksey Shipilev <shade at openjdk.org> wrote:

> When debugging CDS, I asked for `os::print_location` before OopStorage was initialized, like an error handler would do. This is a fairly unusual situation, this is why we have not seen it before. Anyhow, we entered the new code added by [JDK-8340392](https://bugs.openjdk.org/browse/JDK-8340392), which crashed on `OopStorage` that was `nullptr`. I think we should null-check `OopStorage` before calling into it.
> 
> Additional testing:
>  - [x] OopStorageSetTest still passing
>  - [x] Verified the check is now passing in similar debugging session

This pull request has now been integrated.

Changeset: 6587909c
Author:    Aleksey Shipilev <shade at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/6587909c7db6482bda92d314096a2a1795900ffd
Stats:     3 lines in 1 file changed: 2 ins; 0 del; 1 mod

8341015: OopStorage location decoder crashes accessing non-initalized OopStorage

Reviewed-by: kbarrett, tschatzl

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

PR: https://git.openjdk.org/jdk/pull/21204