Integrated: 8366196: Crash in AbstractMethodCheck.getMethodCheckFailure

[Jan Lahoda]

On Thu, 30 Oct 2025 16:47:56 GMT, Jan Lahoda <jlahoda at openjdk.org> wrote:

> `AttrRecover` invokes `Attr.checkMethod` with a cached `CheckContext`. `checkMethod` may be calling `report` on the `CheckContext`:
> https://github.com/openjdk/jdk/blob/ed36b9bb6f3d429db6accfb3b096e50e7f2217ff/src/jdk.compiler/share/classes/com/sun/tools/javac/comp/Attr.java#L4986
> 
> This fails if the cached `CheckContext` is a `MethodCheckContext`, whose `report` method throws.
> 
> The proposal herein is to not delegate to the cached `CheckContext`'s `report` method, but rather always report the error using `Check.basicHandler`.

This pull request has now been integrated.

Changeset: 15fd5292
Author:    Jan Lahoda <jlahoda at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/15fd52925ab64a6a9e33691eed9e952a7b75da4b
Stats:     66 lines in 2 files changed: 64 ins; 0 del; 2 mod

8366196: Crash in AbstractMethodCheck.getMethodCheckFailure

Reviewed-by: liach, mcimadamore

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

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