RFR: 8369838: Likely invalid assert or function call in jimage.cpp [v2]

[David Holmes]

On Tue, 4 Nov 2025 17:00:02 GMT, Henry Jen <henryjen at openjdk.org> wrote:

>> JVM should not attempt to load resource without module name. All classes loaded from jimage by the bootclassloader should have mapped into a module.
>
> Henry Jen has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Explicitly initialize location

I dug into the history of this but it was inconclusive. The correctness of the change boils down to the claim that the call with "" can never succeed. I don't know enough about the jimage file to verify that, but looking at the original commit there is nothing to suggest an empty module name is expected/valid, so I will defer to the testing that has been done here.

It might be worthwhile experimenting with JDK 9 to see if something was different back then.

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

Marked as reviewed by dholmes (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/28098#pullrequestreview-3484824542