RFR: 8372787: ModulePatcher$PatchedModuleReader throws unspecified IllegalStateException upon being invoked after close()

[Alan Bateman]

On Mon, 1 Dec 2025 06:08:00 GMT, Jaikiran Pai <jpai at openjdk.org> wrote:

> Can I please get a review of this change which proposes to address the issue noted in https://bugs.openjdk.org/browse/JDK-8372787?
> 
> The commit in this PR takes into account the `IllegalStateException` thrown by `JarFile` APIs and wraps them into a `IOException` to conform with the expectations of the `ModuleReader` APIs.
> 
> A new jtreg test has been introduced to reproduce the issue and verify the fix. CI testing is currently in progress with this change.

@jaikiran Would it be possible to update the bug with a summary as to how this arise when using --patch-module? The proposed test is using the internal ModulePatcher directly and I think the starting point for discussion is an outline as to how anything using --patch-module runs into this.

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

PR Comment: https://git.openjdk.org/jdk/pull/28569#issuecomment-3594856116