Skip to content

8362306: HotSpotJVMCIRuntime.getMirror can crash #26346

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 6 commits into from
Closed

8362306: HotSpotJVMCIRuntime.getMirror can crash #26346

Changes from 1 commit
Commits
Show all changes
6 commits
Select commit Hold shift + click to select a range
cca137a
fixed bug in HotSpotJVMCIRuntime.getMirror for <clinit> and injected …
dougxc Jul 16, 2025
c2c41aa
moved inject field check into check_field for benefit of get_encoded_…
dougxc Jul 16, 2025
c76efe5
fixed bug in use of JVMCI traps
dougxc Jul 16, 2025
7cf4b26
revert from JVMCI_CHECK_NULL back to JVMCIENV in final return stateme…
dougxc Jul 16, 2025
fd44d95
test that getMirror on clinit returns null
dougxc Jul 16, 2025
d62c74d
replaced use of JVMCIENV with JVMCI_CHECK_NULL in getDeclaredFieldsInfo
dougxc Jul 17, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
test that getMirror on clinit returns null
  • Loading branch information
@dougxc
dougxc committed Jul 16, 2025
commit fd44d95c83bb9160395b5fe94298f1a300233001
1 change: 1 addition & 0 deletions ...ompiler/jvmci/jdk.vm.ci.runtime.test/src/jdk/vm/ci/runtime/test/TestResolvedJavaType.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1110,6 +1110,7 @@ private static ResolvedJavaMethod getClassInitializer(Class<?> c) {
if (clinit != null) {
assertEquals(0, clinit.getAnnotations().length);
assertEquals(0, clinit.getDeclaredAnnotations().length);
assertNull(runtime.getMirror(clinit));
}
return clinit;
}
Expand Down