Prepared by: | Andreas Schoesser on Fri Jul 18 09:05:53 CEST 2014 |
---|---|
Workspace: | /net/usr.work/d051827/OpenJDK/hs-comp/hotspot |
Compare against: | http://hg.openjdk.java.net/jdk9/hs-comp/hotspot |
Compare against version: | 6727 |
Summary of changes: | 11 lines changed: 11 ins; 0 del; 0 mod; 392 unchg |
Changeset: | hotspot.changeset |
Legend: |
Modified file Deleted file New file |
Cdiffs
Udiffs
Wdiffs
Sdiffs
Frames
Old
New
-----
Raw
src/share/vm/ci/ciField.cpp
rev 6728 : 8050978: Fix bad field access check in C1 and C2 Summary: JCK8 test vm/constantpool/accessControl/accessControl004/accessControl00402m3/accessControl00402m3.html fails with -Xbatch -Xcomp due to bad field access check in C1 and C2. Fix: In ciField::ciField(), just before the canonical holder is stored into the _holder variable (and which is used by ciField::will_link()) perform an additional access check with the holder declared in the class file. If this check fails, store the declared holder instead and ciField::will_link() will bail out compilation for this field later on. Then, the interpreter will throw an PrivilegedAccessException at runtime. Contributed-by: andreas.schoesser@sap.com11 lines changed: 11 ins; 0 del; 0 mod; 392 unchg
This code review page was prepared using /sapmnt/home1/d051827/myscripts/webrev.ksh (vers 25.5-hg+openjdk.java.net).