RFR - 8065552: setAccessible(true) on fields of Class may throw a SecurityException

Mandy Chung mandy.chung at oracle.com
Mon Dec 1 18:50:32 UTC 2014


On 12/1/14 8:29 AM, Daniel Fuchs wrote:
> 8065552: setAccessible(true) on fields of Class may throw
>          a SecurityException
>
> webrev:
> http://cr.openjdk.java.net/~dfuchs/webrev_8065552/webrev.00/ 

Thanks for taking this on.  Looks okay in general.  The comment on 
Class.classLoader field - may simply say "this field is filtered from 
reflection access, i.e. getDeclaredField will throw NoSuchFieldException".

ClassDeclaredFieldsTest.java
    copyright year should be 2014.
    @summary - this test also verifies that Class.classLoader final 
private field is hidden from reflection access.
    line 86: nit:  can just call "new ThreadLocal<>"

Mandy



More information about the core-libs-dev mailing list