stan/jdk-24
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

8076236: VM permits illegal flags for class init method

Browse Source 
Move check for multiple access flags so that it also covers instance initialization methods

Reviewed-by: ctornqvi, lfoltan
This commit is contained in:
Harold Seigel 2015-04-02 08:50:10 -04:00
parent afa2dbda46
commit 64bb2ce311
1 changed files with 13 additions and 12 deletions

25
hotspot/src/share/vm/classfile/classFileParser.cpp

@ -4838,20 +4838,21 @@ void ClassFileParser::verify_legal_method_modifiers(
}
}
} else { // not interface
if (is_initializer) {
if (is_static || is_final || is_synchronized || is_native ||
is_abstract || (major_gte_15 && is_bridge)) {
is_illegal = true;
}
} else { // not initializer
if (is_abstract) {
if ((is_final || is_native || is_private || is_static ||
(major_gte_15 && (is_synchronized || is_strict)))) {
if (has_illegal_visibility(flags)) {
is_illegal = true;
} else {
if (is_initializer) {
if (is_static || is_final || is_synchronized || is_native ||
is_abstract || (major_gte_15 && is_bridge)) {
is_illegal = true;
}
}
if (has_illegal_visibility(flags)) {
is_illegal = true;
} else { // not initializer
if (is_abstract) {
if ((is_final || is_native || is_private || is_static ||
(major_gte_15 && (is_synchronized || is_strict)))) {
is_illegal = true;
}
}
}
}
}