Mercurial > hg > graal-jvmci-8
diff src/share/vm/runtime/reflection.cpp @ 14204:866fafe5cc33
8027458: VM anonymous classes: wrong context for protected access checks
Summary: Use the anonymous class's host class for protected access checks
Reviewed-by: acorn, coleenp, lfoltan
author | hseigel |
---|---|
date | Wed, 04 Dec 2013 08:10:31 -0500 |
parents | cdf20166ec45 |
children | 63a4eb8bcd23 2373a1f4987c |
line wrap: on
line diff
--- a/src/share/vm/runtime/reflection.cpp Tue Dec 31 19:26:57 2013 +0400 +++ b/src/share/vm/runtime/reflection.cpp Wed Dec 04 08:10:31 2013 -0500 @@ -537,15 +537,26 @@ return true; } + Klass* host_class = current_class; + while (host_class->oop_is_instance() && + InstanceKlass::cast(host_class)->is_anonymous()) { + Klass* next_host_class = InstanceKlass::cast(host_class)->host_klass(); + if (next_host_class == NULL) break; + host_class = next_host_class; + } + if (host_class == field_class) { + return true; + } + if (access.is_protected()) { if (!protected_restriction) { - // See if current_class is a subclass of field_class - if (current_class->is_subclass_of(field_class)) { + // See if current_class (or outermost host class) is a subclass of field_class + if (host_class->is_subclass_of(field_class)) { if (access.is_static() || // static fields are ok, see 6622385 current_class == resolved_class || field_class == resolved_class || - current_class->is_subclass_of(resolved_class) || - resolved_class->is_subclass_of(current_class)) { + host_class->is_subclass_of(resolved_class) || + resolved_class->is_subclass_of(host_class)) { return true; } }