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

[Alan Bateman]

On Sat, 1 Nov 2025 15:42:37 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.

Would it be possible to update the PR with a brief summary on what testing has been done, or just confirm that at least tier1-5 has been run.  It would indeed be a bug if JImageFindResource were called with a zero-length module name so just want to make sure the assert doesn't trigger in some unusual configuration.

As regards asserts vs. JIMAGE_* functions returning errors then I assume errors would be preferable. The classes in classLoader.cpp is the only user so it wouldn't be hard to audit the usage to make sure that they check for errors.

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

PR Comment: https://git.openjdk.org/jdk/pull/28098#issuecomment-3477557140