src/hotspot/share/runtime/reflection.cpp
author coleenp
Wed, 19 Sep 2018 15:25:05 -0400
changeset 51807 3221f5e14866
parent 51467 12997ebbc0d8
child 52096 7a1e2d7ac55a
permissions -rw-r--r--
8198717: Remove compute_optional_offset Summary: remove optional_offset computation and related unused code from javaClasses for reflection. Reviewed-by: redestad, lfoltan
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
     1
/*
49359
59f6547e151f 8199264: Remove universe.inline.hpp to simplify include dependencies
stefank
parents: 49036
diff changeset
     2
 * Copyright (c) 1997, 2018, Oracle and/or its affiliates. All rights reserved.
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
     3
 * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
489c9b5090e2 Initial load
duke
parents:
diff changeset
     4
 *
489c9b5090e2 Initial load
duke
parents:
diff changeset
     5
 * This code is free software; you can redistribute it and/or modify it
489c9b5090e2 Initial load
duke
parents:
diff changeset
     6
 * under the terms of the GNU General Public License version 2 only, as
489c9b5090e2 Initial load
duke
parents:
diff changeset
     7
 * published by the Free Software Foundation.
489c9b5090e2 Initial load
duke
parents:
diff changeset
     8
 *
489c9b5090e2 Initial load
duke
parents:
diff changeset
     9
 * This code is distributed in the hope that it will be useful, but WITHOUT
489c9b5090e2 Initial load
duke
parents:
diff changeset
    10
 * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
489c9b5090e2 Initial load
duke
parents:
diff changeset
    11
 * FITNESS FOR A PARTICULAR PURPOSE.  See the GNU General Public License
489c9b5090e2 Initial load
duke
parents:
diff changeset
    12
 * version 2 for more details (a copy is included in the LICENSE file that
489c9b5090e2 Initial load
duke
parents:
diff changeset
    13
 * accompanied this code).
489c9b5090e2 Initial load
duke
parents:
diff changeset
    14
 *
489c9b5090e2 Initial load
duke
parents:
diff changeset
    15
 * You should have received a copy of the GNU General Public License version
489c9b5090e2 Initial load
duke
parents:
diff changeset
    16
 * 2 along with this work; if not, write to the Free Software Foundation,
489c9b5090e2 Initial load
duke
parents:
diff changeset
    17
 * Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA.
489c9b5090e2 Initial load
duke
parents:
diff changeset
    18
 *
5547
f4b087cbb361 6941466: Oracle rebranding changes for Hotspot repositories
trims
parents: 4581
diff changeset
    19
 * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA
f4b087cbb361 6941466: Oracle rebranding changes for Hotspot repositories
trims
parents: 4581
diff changeset
    20
 * or visit www.oracle.com if you need additional information or have any
f4b087cbb361 6941466: Oracle rebranding changes for Hotspot repositories
trims
parents: 4581
diff changeset
    21
 * questions.
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    22
 *
489c9b5090e2 Initial load
duke
parents:
diff changeset
    23
 */
489c9b5090e2 Initial load
duke
parents:
diff changeset
    24
7397
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5547
diff changeset
    25
#include "precompiled.hpp"
47765
b7c7428eaab9 8189610: Reconcile jvm.h and all jvm_md.h between java.base and hotspot
coleenp
parents: 47216
diff changeset
    26
#include "jvm.h"
7397
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5547
diff changeset
    27
#include "classfile/javaClasses.hpp"
36508
5f9eee6b383b 8142968: Module System implementation
alanb
parents: 35900
diff changeset
    28
#include "classfile/moduleEntry.hpp"
5f9eee6b383b 8142968: Module System implementation
alanb
parents: 35900
diff changeset
    29
#include "classfile/packageEntry.hpp"
24426
0a69c8cdfca9 8038654: Separate SymbolTable and StringTable code
gziemski
parents: 22897
diff changeset
    30
#include "classfile/stringTable.hpp"
7397
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5547
diff changeset
    31
#include "classfile/systemDictionary.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5547
diff changeset
    32
#include "classfile/verifier.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5547
diff changeset
    33
#include "classfile/vmSymbols.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5547
diff changeset
    34
#include "interpreter/linkResolver.hpp"
49982
9042ffe5b7fe 8200729: Conditional compilation of GCs
stefank
parents: 49658
diff changeset
    35
#include "logging/log.hpp"
7397
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5547
diff changeset
    36
#include "memory/oopFactory.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5547
diff changeset
    37
#include "memory/resourceArea.hpp"
49359
59f6547e151f 8199264: Remove universe.inline.hpp to simplify include dependencies
stefank
parents: 49036
diff changeset
    38
#include "memory/universe.hpp"
7397
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5547
diff changeset
    39
#include "oops/instanceKlass.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5547
diff changeset
    40
#include "oops/objArrayKlass.hpp"
29081
c61eb4914428 8072911: Remove includes of oop.inline.hpp from .hpp files
stefank
parents: 27685
diff changeset
    41
#include "oops/objArrayOop.inline.hpp"
c61eb4914428 8072911: Remove includes of oop.inline.hpp from .hpp files
stefank
parents: 27685
diff changeset
    42
#include "oops/oop.inline.hpp"
49036
bc92debe57e4 8197999: Accessors in typeArrayOopDesc should use new Access API
rkennke
parents: 48836
diff changeset
    43
#include "oops/typeArrayOop.inline.hpp"
22897
77e1024a4a14 4505697: nsk/jdi/ExceptionEvent/_itself_/exevent006 and exevent008 tests fail with InvocationTargetException
jbachorik
parents: 22551
diff changeset
    44
#include "prims/jvmtiExport.hpp"
7397
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5547
diff changeset
    45
#include "runtime/arguments.hpp"
51467
12997ebbc0d8 8209647: constantPoolHandle::constantPoolHandle(ConstantPool*) when precompiled header is disabled
iklam
parents: 51444
diff changeset
    46
#include "runtime/fieldDescriptor.inline.hpp"
7397
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5547
diff changeset
    47
#include "runtime/handles.inline.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5547
diff changeset
    48
#include "runtime/javaCalls.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5547
diff changeset
    49
#include "runtime/reflection.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5547
diff changeset
    50
#include "runtime/reflectionUtils.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5547
diff changeset
    51
#include "runtime/signature.hpp"
49480
d7df2dd501ce 8199809: Don't include frame.inline.hpp and other.inline.hpp from .hpp files
coleenp
parents: 49359
diff changeset
    52
#include "runtime/vframe.inline.hpp"
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    53
34666
1c7168ea0034 8140485: Class load and creation cleanup
mgronlun
parents: 34238
diff changeset
    54
static void trace_class_resolution(const Klass* to_class) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    55
  ResourceMark rm;
489c9b5090e2 Initial load
duke
parents:
diff changeset
    56
  int line_number = -1;
489c9b5090e2 Initial load
duke
parents:
diff changeset
    57
  const char * source_file = NULL;
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 13391
diff changeset
    58
  Klass* caller = NULL;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    59
  JavaThread* jthread = JavaThread::current();
489c9b5090e2 Initial load
duke
parents:
diff changeset
    60
  if (jthread->has_last_Java_frame()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
    61
    vframeStream vfst(jthread);
489c9b5090e2 Initial load
duke
parents:
diff changeset
    62
    // skip over any frames belonging to java.lang.Class
489c9b5090e2 Initial load
duke
parents:
diff changeset
    63
    while (!vfst.at_end() &&
14391
df0a1573d5bd 8000725: NPG: method_holder() and pool_holder() and pool_holder field should be InstanceKlass
coleenp
parents: 14385
diff changeset
    64
           vfst.method()->method_holder()->name() == vmSymbols::java_lang_Class()) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    65
      vfst.next();
489c9b5090e2 Initial load
duke
parents:
diff changeset
    66
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
    67
    if (!vfst.at_end()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
    68
      // this frame is a likely suspect
489c9b5090e2 Initial load
duke
parents:
diff changeset
    69
      caller = vfst.method()->method_holder();
489c9b5090e2 Initial load
duke
parents:
diff changeset
    70
      line_number = vfst.method()->line_number_from_bci(vfst.bci());
14391
df0a1573d5bd 8000725: NPG: method_holder() and pool_holder() and pool_holder field should be InstanceKlass
coleenp
parents: 14385
diff changeset
    71
      Symbol* s = vfst.method()->method_holder()->source_file_name();
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    72
      if (s != NULL) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
    73
        source_file = s->as_C_string();
489c9b5090e2 Initial load
duke
parents:
diff changeset
    74
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
    75
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
    76
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
    77
  if (caller != NULL) {
14488
ab48109f7d1b 8001471: Klass::cast() does nothing
hseigel
parents: 14391
diff changeset
    78
    const char * from = caller->external_name();
ab48109f7d1b 8001471: Klass::cast() does nothing
hseigel
parents: 14391
diff changeset
    79
    const char * to = to_class->external_name();
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    80
    // print in a single call to reduce interleaving between threads
489c9b5090e2 Initial load
duke
parents:
diff changeset
    81
    if (source_file != NULL) {
38151
fffedc5e5cf8 8154110: Update class* and safepoint* logging subsystems
mockner
parents: 38094
diff changeset
    82
      log_debug(class, resolve)("%s %s %s:%d (reflection)", from, to, source_file, line_number);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    83
    } else {
38151
fffedc5e5cf8 8154110: Update class* and safepoint* logging subsystems
mockner
parents: 38094
diff changeset
    84
      log_debug(class, resolve)("%s %s (reflection)", from, to);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    85
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
    86
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
    87
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
    88
489c9b5090e2 Initial load
duke
parents:
diff changeset
    89
489c9b5090e2 Initial load
duke
parents:
diff changeset
    90
oop Reflection::box(jvalue* value, BasicType type, TRAPS) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
    91
  if (type == T_VOID) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
    92
    return NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
    93
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
    94
  if (type == T_OBJECT || type == T_ARRAY) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
    95
    // regular objects are not boxed
489c9b5090e2 Initial load
duke
parents:
diff changeset
    96
    return (oop) value->l;
489c9b5090e2 Initial load
duke
parents:
diff changeset
    97
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
    98
  oop result = java_lang_boxing_object::create(type, value, CHECK_NULL);
489c9b5090e2 Initial load
duke
parents:
diff changeset
    99
  if (result == NULL) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   100
    THROW_(vmSymbols::java_lang_IllegalArgumentException(), result);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   101
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   102
  return result;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   103
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   104
489c9b5090e2 Initial load
duke
parents:
diff changeset
   105
489c9b5090e2 Initial load
duke
parents:
diff changeset
   106
BasicType Reflection::unbox_for_primitive(oop box, jvalue* value, TRAPS) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   107
  if (box == NULL) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   108
    THROW_(vmSymbols::java_lang_IllegalArgumentException(), T_ILLEGAL);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   109
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   110
  return java_lang_boxing_object::get_value(box, value);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   111
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   112
489c9b5090e2 Initial load
duke
parents:
diff changeset
   113
BasicType Reflection::unbox_for_regular_object(oop box, jvalue* value) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   114
  // Note:  box is really the unboxed oop.  It might even be a Short, etc.!
489c9b5090e2 Initial load
duke
parents:
diff changeset
   115
  value->l = (jobject) box;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   116
  return T_OBJECT;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   117
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   118
489c9b5090e2 Initial load
duke
parents:
diff changeset
   119
489c9b5090e2 Initial load
duke
parents:
diff changeset
   120
void Reflection::widen(jvalue* value, BasicType current_type, BasicType wide_type, TRAPS) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   121
  assert(wide_type != current_type, "widen should not be called with identical types");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   122
  switch (wide_type) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   123
    case T_BOOLEAN:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   124
    case T_BYTE:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   125
    case T_CHAR:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   126
      break;  // fail
489c9b5090e2 Initial load
duke
parents:
diff changeset
   127
    case T_SHORT:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   128
      switch (current_type) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   129
        case T_BYTE:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   130
          value->s = (jshort) value->b;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   131
          return;
46630
75aa3e39d02c 8182299: Enable disabled clang warnings, build on OSX 10 + Xcode 8
jwilhelm
parents: 46543
diff changeset
   132
        default:
75aa3e39d02c 8182299: Enable disabled clang warnings, build on OSX 10 + Xcode 8
jwilhelm
parents: 46543
diff changeset
   133
          break;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   134
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   135
      break;  // fail
489c9b5090e2 Initial load
duke
parents:
diff changeset
   136
    case T_INT:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   137
      switch (current_type) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   138
        case T_BYTE:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   139
          value->i = (jint) value->b;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   140
          return;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   141
        case T_CHAR:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   142
          value->i = (jint) value->c;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   143
          return;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   144
        case T_SHORT:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   145
          value->i = (jint) value->s;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   146
          return;
46630
75aa3e39d02c 8182299: Enable disabled clang warnings, build on OSX 10 + Xcode 8
jwilhelm
parents: 46543
diff changeset
   147
        default:
75aa3e39d02c 8182299: Enable disabled clang warnings, build on OSX 10 + Xcode 8
jwilhelm
parents: 46543
diff changeset
   148
          break;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   149
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   150
      break;  // fail
489c9b5090e2 Initial load
duke
parents:
diff changeset
   151
    case T_LONG:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   152
      switch (current_type) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   153
        case T_BYTE:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   154
          value->j = (jlong) value->b;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   155
          return;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   156
        case T_CHAR:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   157
          value->j = (jlong) value->c;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   158
          return;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   159
        case T_SHORT:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   160
          value->j = (jlong) value->s;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   161
          return;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   162
        case T_INT:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   163
          value->j = (jlong) value->i;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   164
          return;
46630
75aa3e39d02c 8182299: Enable disabled clang warnings, build on OSX 10 + Xcode 8
jwilhelm
parents: 46543
diff changeset
   165
        default:
75aa3e39d02c 8182299: Enable disabled clang warnings, build on OSX 10 + Xcode 8
jwilhelm
parents: 46543
diff changeset
   166
          break;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   167
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   168
      break;  // fail
489c9b5090e2 Initial load
duke
parents:
diff changeset
   169
    case T_FLOAT:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   170
      switch (current_type) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   171
        case T_BYTE:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   172
          value->f = (jfloat) value->b;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   173
          return;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   174
        case T_CHAR:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   175
          value->f = (jfloat) value->c;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   176
          return;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   177
        case T_SHORT:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   178
          value->f = (jfloat) value->s;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   179
          return;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   180
        case T_INT:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   181
          value->f = (jfloat) value->i;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   182
          return;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   183
        case T_LONG:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   184
          value->f = (jfloat) value->j;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   185
          return;
46630
75aa3e39d02c 8182299: Enable disabled clang warnings, build on OSX 10 + Xcode 8
jwilhelm
parents: 46543
diff changeset
   186
        default:
75aa3e39d02c 8182299: Enable disabled clang warnings, build on OSX 10 + Xcode 8
jwilhelm
parents: 46543
diff changeset
   187
          break;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   188
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   189
      break;  // fail
489c9b5090e2 Initial load
duke
parents:
diff changeset
   190
    case T_DOUBLE:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   191
      switch (current_type) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   192
        case T_BYTE:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   193
          value->d = (jdouble) value->b;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   194
          return;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   195
        case T_CHAR:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   196
          value->d = (jdouble) value->c;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   197
          return;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   198
        case T_SHORT:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   199
          value->d = (jdouble) value->s;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   200
          return;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   201
        case T_INT:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   202
          value->d = (jdouble) value->i;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   203
          return;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   204
        case T_FLOAT:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   205
          value->d = (jdouble) value->f;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   206
          return;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   207
        case T_LONG:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   208
          value->d = (jdouble) value->j;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   209
          return;
46630
75aa3e39d02c 8182299: Enable disabled clang warnings, build on OSX 10 + Xcode 8
jwilhelm
parents: 46543
diff changeset
   210
        default:
75aa3e39d02c 8182299: Enable disabled clang warnings, build on OSX 10 + Xcode 8
jwilhelm
parents: 46543
diff changeset
   211
          break;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   212
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   213
      break;  // fail
489c9b5090e2 Initial load
duke
parents:
diff changeset
   214
    default:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   215
      break;  // fail
489c9b5090e2 Initial load
duke
parents:
diff changeset
   216
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   217
  THROW_MSG(vmSymbols::java_lang_IllegalArgumentException(), "argument type mismatch");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   218
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   219
489c9b5090e2 Initial load
duke
parents:
diff changeset
   220
489c9b5090e2 Initial load
duke
parents:
diff changeset
   221
BasicType Reflection::array_get(jvalue* value, arrayOop a, int index, TRAPS) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   222
  if (!a->is_within_bounds(index)) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   223
    THROW_(vmSymbols::java_lang_ArrayIndexOutOfBoundsException(), T_ILLEGAL);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   224
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   225
  if (a->is_objArray()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   226
    value->l = (jobject) objArrayOop(a)->obj_at(index);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   227
    return T_OBJECT;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   228
  } else {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   229
    assert(a->is_typeArray(), "just checking");
13952
e3cf184080bc 8000213: NPG: Should have renamed arrayKlass and typeArrayKlass
coleenp
parents: 13728
diff changeset
   230
    BasicType type = TypeArrayKlass::cast(a->klass())->element_type();
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   231
    switch (type) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   232
      case T_BOOLEAN:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   233
        value->z = typeArrayOop(a)->bool_at(index);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   234
        break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   235
      case T_CHAR:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   236
        value->c = typeArrayOop(a)->char_at(index);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   237
        break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   238
      case T_FLOAT:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   239
        value->f = typeArrayOop(a)->float_at(index);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   240
        break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   241
      case T_DOUBLE:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   242
        value->d = typeArrayOop(a)->double_at(index);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   243
        break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   244
      case T_BYTE:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   245
        value->b = typeArrayOop(a)->byte_at(index);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   246
        break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   247
      case T_SHORT:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   248
        value->s = typeArrayOop(a)->short_at(index);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   249
        break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   250
      case T_INT:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   251
        value->i = typeArrayOop(a)->int_at(index);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   252
        break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   253
      case T_LONG:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   254
        value->j = typeArrayOop(a)->long_at(index);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   255
        break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   256
      default:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   257
        return T_ILLEGAL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   258
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   259
    return type;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   260
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   261
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   262
489c9b5090e2 Initial load
duke
parents:
diff changeset
   263
489c9b5090e2 Initial load
duke
parents:
diff changeset
   264
void Reflection::array_set(jvalue* value, arrayOop a, int index, BasicType value_type, TRAPS) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   265
  if (!a->is_within_bounds(index)) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   266
    THROW(vmSymbols::java_lang_ArrayIndexOutOfBoundsException());
489c9b5090e2 Initial load
duke
parents:
diff changeset
   267
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   268
  if (a->is_objArray()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   269
    if (value_type == T_OBJECT) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   270
      oop obj = (oop) value->l;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   271
      if (obj != NULL) {
13952
e3cf184080bc 8000213: NPG: Should have renamed arrayKlass and typeArrayKlass
coleenp
parents: 13728
diff changeset
   272
        Klass* element_klass = ObjArrayKlass::cast(a->klass())->element_klass();
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   273
        if (!obj->is_a(element_klass)) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   274
          THROW_MSG(vmSymbols::java_lang_IllegalArgumentException(), "array element type mismatch");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   275
        }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   276
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   277
      objArrayOop(a)->obj_at_put(index, obj);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   278
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   279
  } else {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   280
    assert(a->is_typeArray(), "just checking");
13952
e3cf184080bc 8000213: NPG: Should have renamed arrayKlass and typeArrayKlass
coleenp
parents: 13728
diff changeset
   281
    BasicType array_type = TypeArrayKlass::cast(a->klass())->element_type();
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   282
    if (array_type != value_type) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   283
      // The widen operation can potentially throw an exception, but cannot block,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   284
      // so typeArrayOop a is safe if the call succeeds.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   285
      widen(value, value_type, array_type, CHECK);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   286
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   287
    switch (array_type) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   288
      case T_BOOLEAN:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   289
        typeArrayOop(a)->bool_at_put(index, value->z);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   290
        break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   291
      case T_CHAR:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   292
        typeArrayOop(a)->char_at_put(index, value->c);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   293
        break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   294
      case T_FLOAT:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   295
        typeArrayOop(a)->float_at_put(index, value->f);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   296
        break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   297
      case T_DOUBLE:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   298
        typeArrayOop(a)->double_at_put(index, value->d);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   299
        break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   300
      case T_BYTE:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   301
        typeArrayOop(a)->byte_at_put(index, value->b);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   302
        break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   303
      case T_SHORT:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   304
        typeArrayOop(a)->short_at_put(index, value->s);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   305
        break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   306
      case T_INT:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   307
        typeArrayOop(a)->int_at_put(index, value->i);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   308
        break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   309
      case T_LONG:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   310
        typeArrayOop(a)->long_at_put(index, value->j);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   311
        break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   312
      default:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   313
        THROW(vmSymbols::java_lang_IllegalArgumentException());
489c9b5090e2 Initial load
duke
parents:
diff changeset
   314
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   315
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   316
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   317
34666
1c7168ea0034 8140485: Class load and creation cleanup
mgronlun
parents: 34238
diff changeset
   318
static Klass* basic_type_mirror_to_arrayklass(oop basic_type_mirror, TRAPS) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   319
  assert(java_lang_Class::is_primitive(basic_type_mirror), "just checking");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   320
  BasicType type = java_lang_Class::primitive_type(basic_type_mirror);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   321
  if (type == T_VOID) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   322
    THROW_0(vmSymbols::java_lang_IllegalArgumentException());
34666
1c7168ea0034 8140485: Class load and creation cleanup
mgronlun
parents: 34238
diff changeset
   323
  }
1c7168ea0034 8140485: Class load and creation cleanup
mgronlun
parents: 34238
diff changeset
   324
  else {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   325
    return Universe::typeArrayKlassObj(type);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   326
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   327
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   328
34666
1c7168ea0034 8140485: Class load and creation cleanup
mgronlun
parents: 34238
diff changeset
   329
#ifdef ASSERT
1c7168ea0034 8140485: Class load and creation cleanup
mgronlun
parents: 34238
diff changeset
   330
static oop basic_type_arrayklass_to_mirror(Klass* basic_type_arrayklass, TRAPS) {
13952
e3cf184080bc 8000213: NPG: Should have renamed arrayKlass and typeArrayKlass
coleenp
parents: 13728
diff changeset
   331
  BasicType type = TypeArrayKlass::cast(basic_type_arrayklass)->element_type();
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   332
  return Universe::java_mirror(type);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   333
}
34666
1c7168ea0034 8140485: Class load and creation cleanup
mgronlun
parents: 34238
diff changeset
   334
#endif
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   335
489c9b5090e2 Initial load
duke
parents:
diff changeset
   336
arrayOop Reflection::reflect_new_array(oop element_mirror, jint length, TRAPS) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   337
  if (element_mirror == NULL) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   338
    THROW_0(vmSymbols::java_lang_NullPointerException());
489c9b5090e2 Initial load
duke
parents:
diff changeset
   339
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   340
  if (length < 0) {
50304
d5331b94f821 8203881: Print erroneous size in NegativeArraySizeException
goetz
parents: 49982
diff changeset
   341
    THROW_MSG_0(vmSymbols::java_lang_NegativeArraySizeException(), err_msg("%d", length));
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   342
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   343
  if (java_lang_Class::is_primitive(element_mirror)) {
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 13391
diff changeset
   344
    Klass* tak = basic_type_mirror_to_arrayklass(element_mirror, CHECK_NULL);
13952
e3cf184080bc 8000213: NPG: Should have renamed arrayKlass and typeArrayKlass
coleenp
parents: 13728
diff changeset
   345
    return TypeArrayKlass::cast(tak)->allocate(length, THREAD);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   346
  } else {
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 13391
diff changeset
   347
    Klass* k = java_lang_Class::as_Klass(element_mirror);
33611
9abd65805e19 8139203: Consistent naming for klass type predicates
coleenp
parents: 33602
diff changeset
   348
    if (k->is_array_klass() && ArrayKlass::cast(k)->dimension() >= MAX_DIM) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   349
      THROW_0(vmSymbols::java_lang_IllegalArgumentException());
489c9b5090e2 Initial load
duke
parents:
diff changeset
   350
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   351
    return oopFactory::new_objArray(k, length, THREAD);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   352
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   353
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   354
489c9b5090e2 Initial load
duke
parents:
diff changeset
   355
489c9b5090e2 Initial load
duke
parents:
diff changeset
   356
arrayOop Reflection::reflect_new_multi_array(oop element_mirror, typeArrayOop dim_array, TRAPS) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   357
  assert(dim_array->is_typeArray(), "just checking");
13952
e3cf184080bc 8000213: NPG: Should have renamed arrayKlass and typeArrayKlass
coleenp
parents: 13728
diff changeset
   358
  assert(TypeArrayKlass::cast(dim_array->klass())->element_type() == T_INT, "just checking");
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   359
489c9b5090e2 Initial load
duke
parents:
diff changeset
   360
  if (element_mirror == NULL) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   361
    THROW_0(vmSymbols::java_lang_NullPointerException());
489c9b5090e2 Initial load
duke
parents:
diff changeset
   362
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   363
489c9b5090e2 Initial load
duke
parents:
diff changeset
   364
  int len = dim_array->length();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   365
  if (len <= 0 || len > MAX_DIM) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   366
    THROW_0(vmSymbols::java_lang_IllegalArgumentException());
489c9b5090e2 Initial load
duke
parents:
diff changeset
   367
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   368
489c9b5090e2 Initial load
duke
parents:
diff changeset
   369
  jint dimensions[MAX_DIM];   // C array copy of intArrayOop
489c9b5090e2 Initial load
duke
parents:
diff changeset
   370
  for (int i = 0; i < len; i++) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   371
    int d = dim_array->int_at(i);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   372
    if (d < 0) {
50304
d5331b94f821 8203881: Print erroneous size in NegativeArraySizeException
goetz
parents: 49982
diff changeset
   373
      THROW_MSG_0(vmSymbols::java_lang_NegativeArraySizeException(), err_msg("%d", d));
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   374
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   375
    dimensions[i] = d;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   376
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   377
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 13391
diff changeset
   378
  Klass* klass;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   379
  int dim = len;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   380
  if (java_lang_Class::is_primitive(element_mirror)) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   381
    klass = basic_type_mirror_to_arrayklass(element_mirror, CHECK_NULL);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   382
  } else {
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 13391
diff changeset
   383
    klass = java_lang_Class::as_Klass(element_mirror);
33611
9abd65805e19 8139203: Consistent naming for klass type predicates
coleenp
parents: 33602
diff changeset
   384
    if (klass->is_array_klass()) {
13952
e3cf184080bc 8000213: NPG: Should have renamed arrayKlass and typeArrayKlass
coleenp
parents: 13728
diff changeset
   385
      int k_dim = ArrayKlass::cast(klass)->dimension();
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   386
      if (k_dim + len > MAX_DIM) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   387
        THROW_0(vmSymbols::java_lang_IllegalArgumentException());
489c9b5090e2 Initial load
duke
parents:
diff changeset
   388
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   389
      dim += k_dim;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   390
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   391
  }
14488
ab48109f7d1b 8001471: Klass::cast() does nothing
hseigel
parents: 14391
diff changeset
   392
  klass = klass->array_klass(dim, CHECK_NULL);
17861
8f14da01157d 6726963: multi_allocate() call does not CHECK_NULL and causes crash in fastdebug bits
ctornqvi
parents: 17860
diff changeset
   393
  oop obj = ArrayKlass::cast(klass)->multi_allocate(len, dimensions, CHECK_NULL);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   394
  assert(obj->is_array(), "just checking");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   395
  return arrayOop(obj);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   396
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   397
489c9b5090e2 Initial load
duke
parents:
diff changeset
   398
489c9b5090e2 Initial load
duke
parents:
diff changeset
   399
oop Reflection::array_component_type(oop mirror, TRAPS) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   400
  if (java_lang_Class::is_primitive(mirror)) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   401
    return NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   402
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   403
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 13391
diff changeset
   404
  Klass* klass = java_lang_Class::as_Klass(mirror);
33611
9abd65805e19 8139203: Consistent naming for klass type predicates
coleenp
parents: 33602
diff changeset
   405
  if (!klass->is_array_klass()) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   406
    return NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   407
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   408
26412
80741eb33ba2 8049105: Move array component mirror to instance of java/lang/Class (hotspot part 2)
coleenp
parents: 25057
diff changeset
   409
  oop result = java_lang_Class::component_mirror(mirror);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   410
#ifdef ASSERT
489c9b5090e2 Initial load
duke
parents:
diff changeset
   411
  oop result2 = NULL;
13952
e3cf184080bc 8000213: NPG: Should have renamed arrayKlass and typeArrayKlass
coleenp
parents: 13728
diff changeset
   412
  if (ArrayKlass::cast(klass)->dimension() == 1) {
33611
9abd65805e19 8139203: Consistent naming for klass type predicates
coleenp
parents: 33602
diff changeset
   413
    if (klass->is_typeArray_klass()) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   414
      result2 = basic_type_arrayklass_to_mirror(klass, CHECK_NULL);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   415
    } else {
14488
ab48109f7d1b 8001471: Klass::cast() does nothing
hseigel
parents: 14391
diff changeset
   416
      result2 = ObjArrayKlass::cast(klass)->element_klass()->java_mirror();
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   417
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   418
  } else {
13952
e3cf184080bc 8000213: NPG: Should have renamed arrayKlass and typeArrayKlass
coleenp
parents: 13728
diff changeset
   419
    Klass* lower_dim = ArrayKlass::cast(klass)->lower_dimension();
33611
9abd65805e19 8139203: Consistent naming for klass type predicates
coleenp
parents: 33602
diff changeset
   420
    assert(lower_dim->is_array_klass(), "just checking");
14488
ab48109f7d1b 8001471: Klass::cast() does nothing
hseigel
parents: 14391
diff changeset
   421
    result2 = lower_dim->java_mirror();
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   422
  }
49658
8237a91c1cca 8199781: Don't use naked == for comparing oops
rkennke
parents: 49480
diff changeset
   423
  assert(oopDesc::equals(result, result2), "results must be consistent");
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   424
#endif //ASSERT
489c9b5090e2 Initial load
duke
parents:
diff changeset
   425
  return result;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   426
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   427
51444
3e5d28e6de32 8209301: JVM rename is_anonymous, host_klass to unsafe specific terminology ahead of Unsafe.defineAnonymousClass deprecation
lfoltan
parents: 50735
diff changeset
   428
static bool under_unsafe_anonymous_host(const InstanceKlass* ik, const InstanceKlass* unsafe_anonymous_host) {
34666
1c7168ea0034 8140485: Class load and creation cleanup
mgronlun
parents: 34238
diff changeset
   429
  DEBUG_ONLY(int inf_loop_check = 1000 * 1000 * 1000);
1c7168ea0034 8140485: Class load and creation cleanup
mgronlun
parents: 34238
diff changeset
   430
  for (;;) {
51444
3e5d28e6de32 8209301: JVM rename is_anonymous, host_klass to unsafe specific terminology ahead of Unsafe.defineAnonymousClass deprecation
lfoltan
parents: 50735
diff changeset
   431
    const InstanceKlass* hc = ik->unsafe_anonymous_host();
34666
1c7168ea0034 8140485: Class load and creation cleanup
mgronlun
parents: 34238
diff changeset
   432
    if (hc == NULL)        return false;
51444
3e5d28e6de32 8209301: JVM rename is_anonymous, host_klass to unsafe specific terminology ahead of Unsafe.defineAnonymousClass deprecation
lfoltan
parents: 50735
diff changeset
   433
    if (hc == unsafe_anonymous_host)  return true;
40923
10fe1c28b9f6 8058575: IllegalAccessError trying to access package-private class from VM anonymous class
hseigel
parents: 40856
diff changeset
   434
    ik = hc;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   435
34666
1c7168ea0034 8140485: Class load and creation cleanup
mgronlun
parents: 34238
diff changeset
   436
    // There's no way to make a host class loop short of patching memory.
1c7168ea0034 8140485: Class load and creation cleanup
mgronlun
parents: 34238
diff changeset
   437
    // Therefore there cannot be a loop here unless there's another bug.
1c7168ea0034 8140485: Class load and creation cleanup
mgronlun
parents: 34238
diff changeset
   438
    // Still, let's check for it.
51444
3e5d28e6de32 8209301: JVM rename is_anonymous, host_klass to unsafe specific terminology ahead of Unsafe.defineAnonymousClass deprecation
lfoltan
parents: 50735
diff changeset
   439
    assert(--inf_loop_check > 0, "no unsafe_anonymous_host loop");
34666
1c7168ea0034 8140485: Class load and creation cleanup
mgronlun
parents: 34238
diff changeset
   440
  }
1c7168ea0034 8140485: Class load and creation cleanup
mgronlun
parents: 34238
diff changeset
   441
}
1c7168ea0034 8140485: Class load and creation cleanup
mgronlun
parents: 34238
diff changeset
   442
1c7168ea0034 8140485: Class load and creation cleanup
mgronlun
parents: 34238
diff changeset
   443
static bool can_relax_access_check_for(const Klass* accessor,
1c7168ea0034 8140485: Class load and creation cleanup
mgronlun
parents: 34238
diff changeset
   444
                                       const Klass* accessee,
1c7168ea0034 8140485: Class load and creation cleanup
mgronlun
parents: 34238
diff changeset
   445
                                       bool classloader_only) {
1c7168ea0034 8140485: Class load and creation cleanup
mgronlun
parents: 34238
diff changeset
   446
1c7168ea0034 8140485: Class load and creation cleanup
mgronlun
parents: 34238
diff changeset
   447
  const InstanceKlass* accessor_ik = InstanceKlass::cast(accessor);
1c7168ea0034 8140485: Class load and creation cleanup
mgronlun
parents: 34238
diff changeset
   448
  const InstanceKlass* accessee_ik = InstanceKlass::cast(accessee);
1c7168ea0034 8140485: Class load and creation cleanup
mgronlun
parents: 34238
diff changeset
   449
51444
3e5d28e6de32 8209301: JVM rename is_anonymous, host_klass to unsafe specific terminology ahead of Unsafe.defineAnonymousClass deprecation
lfoltan
parents: 50735
diff changeset
   450
  // If either is on the other's unsafe_anonymous_host chain, access is OK,
34666
1c7168ea0034 8140485: Class load and creation cleanup
mgronlun
parents: 34238
diff changeset
   451
  // because one is inside the other.
51444
3e5d28e6de32 8209301: JVM rename is_anonymous, host_klass to unsafe specific terminology ahead of Unsafe.defineAnonymousClass deprecation
lfoltan
parents: 50735
diff changeset
   452
  if (under_unsafe_anonymous_host(accessor_ik, accessee_ik) ||
3e5d28e6de32 8209301: JVM rename is_anonymous, host_klass to unsafe specific terminology ahead of Unsafe.defineAnonymousClass deprecation
lfoltan
parents: 50735
diff changeset
   453
    under_unsafe_anonymous_host(accessee_ik, accessor_ik))
34666
1c7168ea0034 8140485: Class load and creation cleanup
mgronlun
parents: 34238
diff changeset
   454
    return true;
1c7168ea0034 8140485: Class load and creation cleanup
mgronlun
parents: 34238
diff changeset
   455
1c7168ea0034 8140485: Class load and creation cleanup
mgronlun
parents: 34238
diff changeset
   456
  if ((RelaxAccessControlCheck &&
1c7168ea0034 8140485: Class load and creation cleanup
mgronlun
parents: 34238
diff changeset
   457
    accessor_ik->major_version() < Verifier::NO_RELAX_ACCESS_CTRL_CHECK_VERSION &&
1c7168ea0034 8140485: Class load and creation cleanup
mgronlun
parents: 34238
diff changeset
   458
    accessee_ik->major_version() < Verifier::NO_RELAX_ACCESS_CTRL_CHECK_VERSION) ||
1c7168ea0034 8140485: Class load and creation cleanup
mgronlun
parents: 34238
diff changeset
   459
    (accessor_ik->major_version() < Verifier::STRICTER_ACCESS_CTRL_CHECK_VERSION &&
1c7168ea0034 8140485: Class load and creation cleanup
mgronlun
parents: 34238
diff changeset
   460
    accessee_ik->major_version() < Verifier::STRICTER_ACCESS_CTRL_CHECK_VERSION)) {
1c7168ea0034 8140485: Class load and creation cleanup
mgronlun
parents: 34238
diff changeset
   461
    return classloader_only &&
40856
3378947a95e6 8148854: Class names "SomeClass" and "LSomeClass;" treated by JVM as an equivalent
rprotacio
parents: 40385
diff changeset
   462
      Verifier::relax_access_for(accessor_ik->class_loader()) &&
34666
1c7168ea0034 8140485: Class load and creation cleanup
mgronlun
parents: 34238
diff changeset
   463
      accessor_ik->protection_domain() == accessee_ik->protection_domain() &&
1c7168ea0034 8140485: Class load and creation cleanup
mgronlun
parents: 34238
diff changeset
   464
      accessor_ik->class_loader() == accessee_ik->class_loader();
1c7168ea0034 8140485: Class load and creation cleanup
mgronlun
parents: 34238
diff changeset
   465
  }
1c7168ea0034 8140485: Class load and creation cleanup
mgronlun
parents: 34238
diff changeset
   466
1c7168ea0034 8140485: Class load and creation cleanup
mgronlun
parents: 34238
diff changeset
   467
  return false;
1c7168ea0034 8140485: Class load and creation cleanup
mgronlun
parents: 34238
diff changeset
   468
}
1c7168ea0034 8140485: Class load and creation cleanup
mgronlun
parents: 34238
diff changeset
   469
36508
5f9eee6b383b 8142968: Module System implementation
alanb
parents: 35900
diff changeset
   470
/*
5f9eee6b383b 8142968: Module System implementation
alanb
parents: 35900
diff changeset
   471
    Type Accessibility check for public types: Callee Type T is accessible to Caller Type S if:
5f9eee6b383b 8142968: Module System implementation
alanb
parents: 35900
diff changeset
   472
5f9eee6b383b 8142968: Module System implementation
alanb
parents: 35900
diff changeset
   473
                        Callee T in             Callee T in package PT,
5f9eee6b383b 8142968: Module System implementation
alanb
parents: 35900
diff changeset
   474
                        unnamed module          runtime module MT
5f9eee6b383b 8142968: Module System implementation
alanb
parents: 35900
diff changeset
   475
 ------------------------------------------------------------------------------------------------
5f9eee6b383b 8142968: Module System implementation
alanb
parents: 35900
diff changeset
   476
5f9eee6b383b 8142968: Module System implementation
alanb
parents: 35900
diff changeset
   477
 Caller S in package     If MS is loose: YES      If same classloader/package (PS == PT): YES
5f9eee6b383b 8142968: Module System implementation
alanb
parents: 35900
diff changeset
   478
 PS, runtime module MS   If MS can read T's       If same runtime module: (MS == MT): YES
5f9eee6b383b 8142968: Module System implementation
alanb
parents: 35900
diff changeset
   479
                         unnamed module: YES
46404
ae62ba99a1a7 8165896: Use "open" flag from JVM_DefineModule to export all module packages
rprotacio
parents: 46388
diff changeset
   480
                                                  Else if (MS can read MT (establish readability) &&
ae62ba99a1a7 8165896: Use "open" flag from JVM_DefineModule to export all module packages
rprotacio
parents: 46388
diff changeset
   481
                                                    ((MT exports PT to MS or to all modules) ||
ae62ba99a1a7 8165896: Use "open" flag from JVM_DefineModule to export all module packages
rprotacio
parents: 46388
diff changeset
   482
                                                     (MT is open))): YES
36508
5f9eee6b383b 8142968: Module System implementation
alanb
parents: 35900
diff changeset
   483
5f9eee6b383b 8142968: Module System implementation
alanb
parents: 35900
diff changeset
   484
 ------------------------------------------------------------------------------------------------
5f9eee6b383b 8142968: Module System implementation
alanb
parents: 35900
diff changeset
   485
 Caller S in unnamed         YES                  Readability exists because unnamed module
5f9eee6b383b 8142968: Module System implementation
alanb
parents: 35900
diff changeset
   486
 module UM                                            "reads" all modules
5f9eee6b383b 8142968: Module System implementation
alanb
parents: 35900
diff changeset
   487
                                                  if (MT exports PT to UM or to all modules): YES
5f9eee6b383b 8142968: Module System implementation
alanb
parents: 35900
diff changeset
   488
5f9eee6b383b 8142968: Module System implementation
alanb
parents: 35900
diff changeset
   489
 ------------------------------------------------------------------------------------------------
5f9eee6b383b 8142968: Module System implementation
alanb
parents: 35900
diff changeset
   490
5f9eee6b383b 8142968: Module System implementation
alanb
parents: 35900
diff changeset
   491
 Note: a loose module is a module that can read all current and future unnamed modules.
5f9eee6b383b 8142968: Module System implementation
alanb
parents: 35900
diff changeset
   492
*/
5f9eee6b383b 8142968: Module System implementation
alanb
parents: 35900
diff changeset
   493
Reflection::VerifyClassAccessResults Reflection::verify_class_access(
46262
83280d968b96 8174092: Remove array-related access checks from Reflection::verify_class_access()
hseigel
parents: 40923
diff changeset
   494
  const Klass* current_class, const InstanceKlass* new_class, bool classloader_only) {
36508
5f9eee6b383b 8142968: Module System implementation
alanb
parents: 35900
diff changeset
   495
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   496
  // Verify that current_class can access new_class.  If the classloader_only
489c9b5090e2 Initial load
duke
parents:
diff changeset
   497
  // flag is set, we automatically allow any accesses in which current_class
489c9b5090e2 Initial load
duke
parents:
diff changeset
   498
  // doesn't have a classloader.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   499
  if ((current_class == NULL) ||
489c9b5090e2 Initial load
duke
parents:
diff changeset
   500
      (current_class == new_class) ||
489c9b5090e2 Initial load
duke
parents:
diff changeset
   501
      is_same_class_package(current_class, new_class)) {
36508
5f9eee6b383b 8142968: Module System implementation
alanb
parents: 35900
diff changeset
   502
    return ACCESS_OK;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   503
  }
37301
a936b4e01afb 8137058: Clear out all non-Critical APIs from sun.reflect
chegar
parents: 37179
diff changeset
   504
  // Allow all accesses from jdk/internal/reflect/MagicAccessorImpl subclasses to
25057
f38210f84f8c 8031819: Remove legacy jdk checks and code
hseigel
parents: 24456
diff changeset
   505
  // succeed trivially.
f38210f84f8c 8031819: Remove legacy jdk checks and code
hseigel
parents: 24456
diff changeset
   506
  if (current_class->is_subclass_of(SystemDictionary::reflect_MagicAccessorImpl_klass())) {
36508
5f9eee6b383b 8142968: Module System implementation
alanb
parents: 35900
diff changeset
   507
    return ACCESS_OK;
5f9eee6b383b 8142968: Module System implementation
alanb
parents: 35900
diff changeset
   508
  }
5f9eee6b383b 8142968: Module System implementation
alanb
parents: 35900
diff changeset
   509
5f9eee6b383b 8142968: Module System implementation
alanb
parents: 35900
diff changeset
   510
  // module boundaries
5f9eee6b383b 8142968: Module System implementation
alanb
parents: 35900
diff changeset
   511
  if (new_class->is_public()) {
5f9eee6b383b 8142968: Module System implementation
alanb
parents: 35900
diff changeset
   512
    // Ignore modules for DumpSharedSpaces because we do not have any package
5f9eee6b383b 8142968: Module System implementation
alanb
parents: 35900
diff changeset
   513
    // or module information for modules other than java.base.
5f9eee6b383b 8142968: Module System implementation
alanb
parents: 35900
diff changeset
   514
    if (DumpSharedSpaces) {
5f9eee6b383b 8142968: Module System implementation
alanb
parents: 35900
diff changeset
   515
      return ACCESS_OK;
5f9eee6b383b 8142968: Module System implementation
alanb
parents: 35900
diff changeset
   516
    }
5f9eee6b383b 8142968: Module System implementation
alanb
parents: 35900
diff changeset
   517
5f9eee6b383b 8142968: Module System implementation
alanb
parents: 35900
diff changeset
   518
    // Find the module entry for current_class, the accessor
38094
46977cd73d86 8152844: JVM InstanceKlass Methods For Obtaining Package/Module Should Be Moved to Klass
rprotacio
parents: 37480
diff changeset
   519
    ModuleEntry* module_from = current_class->module();
36508
5f9eee6b383b 8142968: Module System implementation
alanb
parents: 35900
diff changeset
   520
    // Find the module entry for new_class, the accessee
38094
46977cd73d86 8152844: JVM InstanceKlass Methods For Obtaining Package/Module Should Be Moved to Klass
rprotacio
parents: 37480
diff changeset
   521
    ModuleEntry* module_to = new_class->module();
36508
5f9eee6b383b 8142968: Module System implementation
alanb
parents: 35900
diff changeset
   522
5f9eee6b383b 8142968: Module System implementation
alanb
parents: 35900
diff changeset
   523
    // both in same (possibly unnamed) module
5f9eee6b383b 8142968: Module System implementation
alanb
parents: 35900
diff changeset
   524
    if (module_from == module_to) {
5f9eee6b383b 8142968: Module System implementation
alanb
parents: 35900
diff changeset
   525
      return ACCESS_OK;
5f9eee6b383b 8142968: Module System implementation
alanb
parents: 35900
diff changeset
   526
    }
5f9eee6b383b 8142968: Module System implementation
alanb
parents: 35900
diff changeset
   527
46404
ae62ba99a1a7 8165896: Use "open" flag from JVM_DefineModule to export all module packages
rprotacio
parents: 46388
diff changeset
   528
    // Acceptable access to a type in an unnamed module. Note that since
36508
5f9eee6b383b 8142968: Module System implementation
alanb
parents: 35900
diff changeset
   529
    // unnamed modules can read all unnamed modules, this also handles the
5f9eee6b383b 8142968: Module System implementation
alanb
parents: 35900
diff changeset
   530
    // case where module_from is also unnamed but in a different class loader.
5f9eee6b383b 8142968: Module System implementation
alanb
parents: 35900
diff changeset
   531
    if (!module_to->is_named() &&
5f9eee6b383b 8142968: Module System implementation
alanb
parents: 35900
diff changeset
   532
        (module_from->can_read_all_unnamed() || module_from->can_read(module_to))) {
5f9eee6b383b 8142968: Module System implementation
alanb
parents: 35900
diff changeset
   533
      return ACCESS_OK;
5f9eee6b383b 8142968: Module System implementation
alanb
parents: 35900
diff changeset
   534
    }
5f9eee6b383b 8142968: Module System implementation
alanb
parents: 35900
diff changeset
   535
5f9eee6b383b 8142968: Module System implementation
alanb
parents: 35900
diff changeset
   536
    // Establish readability, check if module_from is allowed to read module_to.
5f9eee6b383b 8142968: Module System implementation
alanb
parents: 35900
diff changeset
   537
    if (!module_from->can_read(module_to)) {
5f9eee6b383b 8142968: Module System implementation
alanb
parents: 35900
diff changeset
   538
      return MODULE_NOT_READABLE;
5f9eee6b383b 8142968: Module System implementation
alanb
parents: 35900
diff changeset
   539
    }
5f9eee6b383b 8142968: Module System implementation
alanb
parents: 35900
diff changeset
   540
46404
ae62ba99a1a7 8165896: Use "open" flag from JVM_DefineModule to export all module packages
rprotacio
parents: 46388
diff changeset
   541
    // Access is allowed if module_to is open, i.e. all its packages are unqualifiedly exported
ae62ba99a1a7 8165896: Use "open" flag from JVM_DefineModule to export all module packages
rprotacio
parents: 46388
diff changeset
   542
    if (module_to->is_open()) {
ae62ba99a1a7 8165896: Use "open" flag from JVM_DefineModule to export all module packages
rprotacio
parents: 46388
diff changeset
   543
      return ACCESS_OK;
ae62ba99a1a7 8165896: Use "open" flag from JVM_DefineModule to export all module packages
rprotacio
parents: 46388
diff changeset
   544
    }
ae62ba99a1a7 8165896: Use "open" flag from JVM_DefineModule to export all module packages
rprotacio
parents: 46388
diff changeset
   545
38094
46977cd73d86 8152844: JVM InstanceKlass Methods For Obtaining Package/Module Should Be Moved to Klass
rprotacio
parents: 37480
diff changeset
   546
    PackageEntry* package_to = new_class->package();
36508
5f9eee6b383b 8142968: Module System implementation
alanb
parents: 35900
diff changeset
   547
    assert(package_to != NULL, "can not obtain new_class' package");
5f9eee6b383b 8142968: Module System implementation
alanb
parents: 35900
diff changeset
   548
43471
bfb383279a16 8171971: Fix timing bug in JVM management of package export lists
hseigel
parents: 40923
diff changeset
   549
    {
bfb383279a16 8171971: Fix timing bug in JVM management of package export lists
hseigel
parents: 40923
diff changeset
   550
      MutexLocker m1(Module_lock);
bfb383279a16 8171971: Fix timing bug in JVM management of package export lists
hseigel
parents: 40923
diff changeset
   551
bfb383279a16 8171971: Fix timing bug in JVM management of package export lists
hseigel
parents: 40923
diff changeset
   552
      // Once readability is established, if module_to exports T unqualifiedly,
bfb383279a16 8171971: Fix timing bug in JVM management of package export lists
hseigel
parents: 40923
diff changeset
   553
      // (to all modules), than whether module_from is in the unnamed module
bfb383279a16 8171971: Fix timing bug in JVM management of package export lists
hseigel
parents: 40923
diff changeset
   554
      // or not does not matter, access is allowed.
bfb383279a16 8171971: Fix timing bug in JVM management of package export lists
hseigel
parents: 40923
diff changeset
   555
      if (package_to->is_unqual_exported()) {
bfb383279a16 8171971: Fix timing bug in JVM management of package export lists
hseigel
parents: 40923
diff changeset
   556
        return ACCESS_OK;
bfb383279a16 8171971: Fix timing bug in JVM management of package export lists
hseigel
parents: 40923
diff changeset
   557
      }
36508
5f9eee6b383b 8142968: Module System implementation
alanb
parents: 35900
diff changeset
   558
43471
bfb383279a16 8171971: Fix timing bug in JVM management of package export lists
hseigel
parents: 40923
diff changeset
   559
      // Access is allowed if both 1 & 2 hold:
bfb383279a16 8171971: Fix timing bug in JVM management of package export lists
hseigel
parents: 40923
diff changeset
   560
      //   1. Readability, module_from can read module_to (established above).
bfb383279a16 8171971: Fix timing bug in JVM management of package export lists
hseigel
parents: 40923
diff changeset
   561
      //   2. Either module_to exports T to module_from qualifiedly.
bfb383279a16 8171971: Fix timing bug in JVM management of package export lists
hseigel
parents: 40923
diff changeset
   562
      //      or
bfb383279a16 8171971: Fix timing bug in JVM management of package export lists
hseigel
parents: 40923
diff changeset
   563
      //      module_to exports T to all unnamed modules and module_from is unnamed.
bfb383279a16 8171971: Fix timing bug in JVM management of package export lists
hseigel
parents: 40923
diff changeset
   564
      //      or
bfb383279a16 8171971: Fix timing bug in JVM management of package export lists
hseigel
parents: 40923
diff changeset
   565
      //      module_to exports T unqualifiedly to all modules (checked above).
bfb383279a16 8171971: Fix timing bug in JVM management of package export lists
hseigel
parents: 40923
diff changeset
   566
      if (!package_to->is_qexported_to(module_from)) {
bfb383279a16 8171971: Fix timing bug in JVM management of package export lists
hseigel
parents: 40923
diff changeset
   567
        return TYPE_NOT_EXPORTED;
bfb383279a16 8171971: Fix timing bug in JVM management of package export lists
hseigel
parents: 40923
diff changeset
   568
      }
36508
5f9eee6b383b 8142968: Module System implementation
alanb
parents: 35900
diff changeset
   569
    }
5f9eee6b383b 8142968: Module System implementation
alanb
parents: 35900
diff changeset
   570
    return ACCESS_OK;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   571
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   572
36508
5f9eee6b383b 8142968: Module System implementation
alanb
parents: 35900
diff changeset
   573
  if (can_relax_access_check_for(current_class, new_class, classloader_only)) {
5f9eee6b383b 8142968: Module System implementation
alanb
parents: 35900
diff changeset
   574
    return ACCESS_OK;
5f9eee6b383b 8142968: Module System implementation
alanb
parents: 35900
diff changeset
   575
  }
5f9eee6b383b 8142968: Module System implementation
alanb
parents: 35900
diff changeset
   576
  return OTHER_PROBLEM;
5f9eee6b383b 8142968: Module System implementation
alanb
parents: 35900
diff changeset
   577
}
5f9eee6b383b 8142968: Module System implementation
alanb
parents: 35900
diff changeset
   578
5f9eee6b383b 8142968: Module System implementation
alanb
parents: 35900
diff changeset
   579
// Return an error message specific to the specified Klass*'s and result.
5f9eee6b383b 8142968: Module System implementation
alanb
parents: 35900
diff changeset
   580
// This function must be called from within a block containing a ResourceMark.
5f9eee6b383b 8142968: Module System implementation
alanb
parents: 35900
diff changeset
   581
char* Reflection::verify_class_access_msg(const Klass* current_class,
46262
83280d968b96 8174092: Remove array-related access checks from Reflection::verify_class_access()
hseigel
parents: 40923
diff changeset
   582
                                          const InstanceKlass* new_class,
46543
a6c67129b3fd 8182089: build error in hotspot/src/share/vm/runtime/reflection.cpp
hseigel
parents: 46404
diff changeset
   583
                                          const VerifyClassAccessResults result) {
36508
5f9eee6b383b 8142968: Module System implementation
alanb
parents: 35900
diff changeset
   584
  assert(result != ACCESS_OK, "must be failure result");
5f9eee6b383b 8142968: Module System implementation
alanb
parents: 35900
diff changeset
   585
  char * msg = NULL;
5f9eee6b383b 8142968: Module System implementation
alanb
parents: 35900
diff changeset
   586
  if (result != OTHER_PROBLEM && new_class != NULL && current_class != NULL) {
5f9eee6b383b 8142968: Module System implementation
alanb
parents: 35900
diff changeset
   587
    // Find the module entry for current_class, the accessor
38094
46977cd73d86 8152844: JVM InstanceKlass Methods For Obtaining Package/Module Should Be Moved to Klass
rprotacio
parents: 37480
diff changeset
   588
    ModuleEntry* module_from = current_class->module();
36508
5f9eee6b383b 8142968: Module System implementation
alanb
parents: 35900
diff changeset
   589
    const char * module_from_name = module_from->is_named() ? module_from->name()->as_C_string() : UNNAMED_MODULE;
5f9eee6b383b 8142968: Module System implementation
alanb
parents: 35900
diff changeset
   590
    const char * current_class_name = current_class->external_name();
5f9eee6b383b 8142968: Module System implementation
alanb
parents: 35900
diff changeset
   591
5f9eee6b383b 8142968: Module System implementation
alanb
parents: 35900
diff changeset
   592
    // Find the module entry for new_class, the accessee
5f9eee6b383b 8142968: Module System implementation
alanb
parents: 35900
diff changeset
   593
    ModuleEntry* module_to = NULL;
38094
46977cd73d86 8152844: JVM InstanceKlass Methods For Obtaining Package/Module Should Be Moved to Klass
rprotacio
parents: 37480
diff changeset
   594
    module_to = new_class->module();
36508
5f9eee6b383b 8142968: Module System implementation
alanb
parents: 35900
diff changeset
   595
    const char * module_to_name = module_to->is_named() ? module_to->name()->as_C_string() : UNNAMED_MODULE;
5f9eee6b383b 8142968: Module System implementation
alanb
parents: 35900
diff changeset
   596
    const char * new_class_name = new_class->external_name();
5f9eee6b383b 8142968: Module System implementation
alanb
parents: 35900
diff changeset
   597
5f9eee6b383b 8142968: Module System implementation
alanb
parents: 35900
diff changeset
   598
    if (result == MODULE_NOT_READABLE) {
5f9eee6b383b 8142968: Module System implementation
alanb
parents: 35900
diff changeset
   599
      assert(module_from->is_named(), "Unnamed modules can read all modules");
5f9eee6b383b 8142968: Module System implementation
alanb
parents: 35900
diff changeset
   600
      if (module_to->is_named()) {
5f9eee6b383b 8142968: Module System implementation
alanb
parents: 35900
diff changeset
   601
        size_t len = 100 + strlen(current_class_name) + 2*strlen(module_from_name) +
5f9eee6b383b 8142968: Module System implementation
alanb
parents: 35900
diff changeset
   602
          strlen(new_class_name) + 2*strlen(module_to_name);
5f9eee6b383b 8142968: Module System implementation
alanb
parents: 35900
diff changeset
   603
        msg = NEW_RESOURCE_ARRAY(char, len);
5f9eee6b383b 8142968: Module System implementation
alanb
parents: 35900
diff changeset
   604
        jio_snprintf(msg, len - 1,
5f9eee6b383b 8142968: Module System implementation
alanb
parents: 35900
diff changeset
   605
          "class %s (in module %s) cannot access class %s (in module %s) because module %s does not read module %s",
5f9eee6b383b 8142968: Module System implementation
alanb
parents: 35900
diff changeset
   606
          current_class_name, module_from_name, new_class_name,
5f9eee6b383b 8142968: Module System implementation
alanb
parents: 35900
diff changeset
   607
          module_to_name, module_from_name, module_to_name);
5f9eee6b383b 8142968: Module System implementation
alanb
parents: 35900
diff changeset
   608
      } else {
46773
fb17cc9a6847 8185717: Make ModuleEntry->module() return an oop not a jobject
hseigel
parents: 46727
diff changeset
   609
        oop jlm = module_to->module();
44520
0553e129e0ec 8177530: Module system implementation refresh (4/2017)
alanb
parents: 43471
diff changeset
   610
        assert(jlm != NULL, "Null jlm in module_to ModuleEntry");
46773
fb17cc9a6847 8185717: Make ModuleEntry->module() return an oop not a jobject
hseigel
parents: 46727
diff changeset
   611
        intptr_t identity_hash = jlm->identity_hash();
36508
5f9eee6b383b 8142968: Module System implementation
alanb
parents: 35900
diff changeset
   612
        size_t len = 160 + strlen(current_class_name) + 2*strlen(module_from_name) +
5f9eee6b383b 8142968: Module System implementation
alanb
parents: 35900
diff changeset
   613
          strlen(new_class_name) + 2*sizeof(uintx);
5f9eee6b383b 8142968: Module System implementation
alanb
parents: 35900
diff changeset
   614
        msg = NEW_RESOURCE_ARRAY(char, len);
5f9eee6b383b 8142968: Module System implementation
alanb
parents: 35900
diff changeset
   615
        jio_snprintf(msg, len - 1,
5f9eee6b383b 8142968: Module System implementation
alanb
parents: 35900
diff changeset
   616
          "class %s (in module %s) cannot access class %s (in unnamed module @" SIZE_FORMAT_HEX ") because module %s does not read unnamed module @" SIZE_FORMAT_HEX,
5f9eee6b383b 8142968: Module System implementation
alanb
parents: 35900
diff changeset
   617
          current_class_name, module_from_name, new_class_name, uintx(identity_hash),
5f9eee6b383b 8142968: Module System implementation
alanb
parents: 35900
diff changeset
   618
          module_from_name, uintx(identity_hash));
5f9eee6b383b 8142968: Module System implementation
alanb
parents: 35900
diff changeset
   619
      }
5f9eee6b383b 8142968: Module System implementation
alanb
parents: 35900
diff changeset
   620
5f9eee6b383b 8142968: Module System implementation
alanb
parents: 35900
diff changeset
   621
    } else if (result == TYPE_NOT_EXPORTED) {
38094
46977cd73d86 8152844: JVM InstanceKlass Methods For Obtaining Package/Module Should Be Moved to Klass
rprotacio
parents: 37480
diff changeset
   622
      assert(new_class->package() != NULL,
36508
5f9eee6b383b 8142968: Module System implementation
alanb
parents: 35900
diff changeset
   623
             "Unnamed packages are always exported");
5f9eee6b383b 8142968: Module System implementation
alanb
parents: 35900
diff changeset
   624
      const char * package_name =
38094
46977cd73d86 8152844: JVM InstanceKlass Methods For Obtaining Package/Module Should Be Moved to Klass
rprotacio
parents: 37480
diff changeset
   625
        new_class->package()->name()->as_klass_external_name();
36508
5f9eee6b383b 8142968: Module System implementation
alanb
parents: 35900
diff changeset
   626
      assert(module_to->is_named(), "Unnamed modules export all packages");
5f9eee6b383b 8142968: Module System implementation
alanb
parents: 35900
diff changeset
   627
      if (module_from->is_named()) {
5f9eee6b383b 8142968: Module System implementation
alanb
parents: 35900
diff changeset
   628
        size_t len = 118 + strlen(current_class_name) + 2*strlen(module_from_name) +
5f9eee6b383b 8142968: Module System implementation
alanb
parents: 35900
diff changeset
   629
          strlen(new_class_name) + 2*strlen(module_to_name) + strlen(package_name);
5f9eee6b383b 8142968: Module System implementation
alanb
parents: 35900
diff changeset
   630
        msg = NEW_RESOURCE_ARRAY(char, len);
5f9eee6b383b 8142968: Module System implementation
alanb
parents: 35900
diff changeset
   631
        jio_snprintf(msg, len - 1,
5f9eee6b383b 8142968: Module System implementation
alanb
parents: 35900
diff changeset
   632
          "class %s (in module %s) cannot access class %s (in module %s) because module %s does not export %s to module %s",
5f9eee6b383b 8142968: Module System implementation
alanb
parents: 35900
diff changeset
   633
          current_class_name, module_from_name, new_class_name,
5f9eee6b383b 8142968: Module System implementation
alanb
parents: 35900
diff changeset
   634
          module_to_name, module_to_name, package_name, module_from_name);
5f9eee6b383b 8142968: Module System implementation
alanb
parents: 35900
diff changeset
   635
      } else {
46773
fb17cc9a6847 8185717: Make ModuleEntry->module() return an oop not a jobject
hseigel
parents: 46727
diff changeset
   636
        oop jlm = module_from->module();
44520
0553e129e0ec 8177530: Module system implementation refresh (4/2017)
alanb
parents: 43471
diff changeset
   637
        assert(jlm != NULL, "Null jlm in module_from ModuleEntry");
46773
fb17cc9a6847 8185717: Make ModuleEntry->module() return an oop not a jobject
hseigel
parents: 46727
diff changeset
   638
        intptr_t identity_hash = jlm->identity_hash();
36508
5f9eee6b383b 8142968: Module System implementation
alanb
parents: 35900
diff changeset
   639
        size_t len = 170 + strlen(current_class_name) + strlen(new_class_name) +
5f9eee6b383b 8142968: Module System implementation
alanb
parents: 35900
diff changeset
   640
          2*strlen(module_to_name) + strlen(package_name) + 2*sizeof(uintx);
5f9eee6b383b 8142968: Module System implementation
alanb
parents: 35900
diff changeset
   641
        msg = NEW_RESOURCE_ARRAY(char, len);
5f9eee6b383b 8142968: Module System implementation
alanb
parents: 35900
diff changeset
   642
        jio_snprintf(msg, len - 1,
5f9eee6b383b 8142968: Module System implementation
alanb
parents: 35900
diff changeset
   643
          "class %s (in unnamed module @" SIZE_FORMAT_HEX ") cannot access class %s (in module %s) because module %s does not export %s to unnamed module @" SIZE_FORMAT_HEX,
5f9eee6b383b 8142968: Module System implementation
alanb
parents: 35900
diff changeset
   644
          current_class_name, uintx(identity_hash), new_class_name, module_to_name,
5f9eee6b383b 8142968: Module System implementation
alanb
parents: 35900
diff changeset
   645
          module_to_name, package_name, uintx(identity_hash));
5f9eee6b383b 8142968: Module System implementation
alanb
parents: 35900
diff changeset
   646
      }
5f9eee6b383b 8142968: Module System implementation
alanb
parents: 35900
diff changeset
   647
    } else {
5f9eee6b383b 8142968: Module System implementation
alanb
parents: 35900
diff changeset
   648
        ShouldNotReachHere();
5f9eee6b383b 8142968: Module System implementation
alanb
parents: 35900
diff changeset
   649
    }
5f9eee6b383b 8142968: Module System implementation
alanb
parents: 35900
diff changeset
   650
  }  // result != OTHER_PROBLEM...
5f9eee6b383b 8142968: Module System implementation
alanb
parents: 35900
diff changeset
   651
  return msg;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   652
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   653
50735
2f2af62dfac7 8010319: Implementation of JEP 181: Nest-Based Access Control
dholmes
parents: 50304
diff changeset
   654
bool Reflection::verify_member_access(const Klass* current_class,
2f2af62dfac7 8010319: Implementation of JEP 181: Nest-Based Access Control
dholmes
parents: 50304
diff changeset
   655
                                      const Klass* resolved_class,
2f2af62dfac7 8010319: Implementation of JEP 181: Nest-Based Access Control
dholmes
parents: 50304
diff changeset
   656
                                      const Klass* member_class,
2f2af62dfac7 8010319: Implementation of JEP 181: Nest-Based Access Control
dholmes
parents: 50304
diff changeset
   657
                                      AccessFlags access,
2f2af62dfac7 8010319: Implementation of JEP 181: Nest-Based Access Control
dholmes
parents: 50304
diff changeset
   658
                                      bool classloader_only,
2f2af62dfac7 8010319: Implementation of JEP 181: Nest-Based Access Control
dholmes
parents: 50304
diff changeset
   659
                                      bool protected_restriction,
2f2af62dfac7 8010319: Implementation of JEP 181: Nest-Based Access Control
dholmes
parents: 50304
diff changeset
   660
                                      TRAPS) {
2f2af62dfac7 8010319: Implementation of JEP 181: Nest-Based Access Control
dholmes
parents: 50304
diff changeset
   661
  // Verify that current_class can access a member of member_class, where that
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   662
  // field's access bits are "access".  We assume that we've already verified
50735
2f2af62dfac7 8010319: Implementation of JEP 181: Nest-Based Access Control
dholmes
parents: 50304
diff changeset
   663
  // that current_class can access member_class.
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   664
  //
489c9b5090e2 Initial load
duke
parents:
diff changeset
   665
  // If the classloader_only flag is set, we automatically allow any accesses
489c9b5090e2 Initial load
duke
parents:
diff changeset
   666
  // in which current_class doesn't have a classloader.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   667
  //
50735
2f2af62dfac7 8010319: Implementation of JEP 181: Nest-Based Access Control
dholmes
parents: 50304
diff changeset
   668
  // "resolved_class" is the runtime type of "member_class". Sometimes we don't
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   669
  // need this distinction (e.g. if all we have is the runtime type, or during
489c9b5090e2 Initial load
duke
parents:
diff changeset
   670
  // class file parsing when we only care about the static type); in that case
50735
2f2af62dfac7 8010319: Implementation of JEP 181: Nest-Based Access Control
dholmes
parents: 50304
diff changeset
   671
  // callers should ensure that resolved_class == member_class.
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   672
  //
489c9b5090e2 Initial load
duke
parents:
diff changeset
   673
  if ((current_class == NULL) ||
50735
2f2af62dfac7 8010319: Implementation of JEP 181: Nest-Based Access Control
dholmes
parents: 50304
diff changeset
   674
      (current_class == member_class) ||
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   675
      access.is_public()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   676
    return true;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   677
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   678
34666
1c7168ea0034 8140485: Class load and creation cleanup
mgronlun
parents: 34238
diff changeset
   679
  const Klass* host_class = current_class;
51444
3e5d28e6de32 8209301: JVM rename is_anonymous, host_klass to unsafe specific terminology ahead of Unsafe.defineAnonymousClass deprecation
lfoltan
parents: 50735
diff changeset
   680
  if (current_class->is_instance_klass() &&
3e5d28e6de32 8209301: JVM rename is_anonymous, host_klass to unsafe specific terminology ahead of Unsafe.defineAnonymousClass deprecation
lfoltan
parents: 50735
diff changeset
   681
      InstanceKlass::cast(current_class)->is_unsafe_anonymous()) {
3e5d28e6de32 8209301: JVM rename is_anonymous, host_klass to unsafe specific terminology ahead of Unsafe.defineAnonymousClass deprecation
lfoltan
parents: 50735
diff changeset
   682
    host_class = InstanceKlass::cast(current_class)->unsafe_anonymous_host();
3e5d28e6de32 8209301: JVM rename is_anonymous, host_klass to unsafe specific terminology ahead of Unsafe.defineAnonymousClass deprecation
lfoltan
parents: 50735
diff changeset
   683
    assert(host_class != NULL, "Unsafe anonymous class has null host class");
40385
a7011a835634 8030221: Checking for anonymous class should check for NULL as well as potential nesting
hseigel
parents: 38719
diff changeset
   684
    assert(!(host_class->is_instance_klass() &&
51444
3e5d28e6de32 8209301: JVM rename is_anonymous, host_klass to unsafe specific terminology ahead of Unsafe.defineAnonymousClass deprecation
lfoltan
parents: 50735
diff changeset
   685
           InstanceKlass::cast(host_class)->is_unsafe_anonymous()),
3e5d28e6de32 8209301: JVM rename is_anonymous, host_klass to unsafe specific terminology ahead of Unsafe.defineAnonymousClass deprecation
lfoltan
parents: 50735
diff changeset
   686
           "unsafe_anonymous_host should not be unsafe anonymous itself");
22215
579f4a1a4aa6 8027458: VM anonymous classes: wrong context for protected access checks
hseigel
parents: 21729
diff changeset
   687
  }
50735
2f2af62dfac7 8010319: Implementation of JEP 181: Nest-Based Access Control
dholmes
parents: 50304
diff changeset
   688
  if (host_class == member_class) {
22215
579f4a1a4aa6 8027458: VM anonymous classes: wrong context for protected access checks
hseigel
parents: 21729
diff changeset
   689
    return true;
579f4a1a4aa6 8027458: VM anonymous classes: wrong context for protected access checks
hseigel
parents: 21729
diff changeset
   690
  }
579f4a1a4aa6 8027458: VM anonymous classes: wrong context for protected access checks
hseigel
parents: 21729
diff changeset
   691
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   692
  if (access.is_protected()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   693
    if (!protected_restriction) {
50735
2f2af62dfac7 8010319: Implementation of JEP 181: Nest-Based Access Control
dholmes
parents: 50304
diff changeset
   694
      // See if current_class (or outermost host class) is a subclass of member_class
27022
2db6fe33afc2 8036533: Method for correct defaults
hseigel
parents: 26412
diff changeset
   695
      // An interface may not access protected members of j.l.Object
50735
2f2af62dfac7 8010319: Implementation of JEP 181: Nest-Based Access Control
dholmes
parents: 50304
diff changeset
   696
      if (!host_class->is_interface() && host_class->is_subclass_of(member_class)) {
362
00cf4bffd828 6622385: Accessing protected static methods
kamg
parents: 224
diff changeset
   697
        if (access.is_static() || // static fields are ok, see 6622385
00cf4bffd828 6622385: Accessing protected static methods
kamg
parents: 224
diff changeset
   698
            current_class == resolved_class ||
50735
2f2af62dfac7 8010319: Implementation of JEP 181: Nest-Based Access Control
dholmes
parents: 50304
diff changeset
   699
            member_class == resolved_class ||
22215
579f4a1a4aa6 8027458: VM anonymous classes: wrong context for protected access checks
hseigel
parents: 21729
diff changeset
   700
            host_class->is_subclass_of(resolved_class) ||
579f4a1a4aa6 8027458: VM anonymous classes: wrong context for protected access checks
hseigel
parents: 21729
diff changeset
   701
            resolved_class->is_subclass_of(host_class)) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   702
          return true;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   703
        }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   704
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   705
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   706
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   707
50735
2f2af62dfac7 8010319: Implementation of JEP 181: Nest-Based Access Control
dholmes
parents: 50304
diff changeset
   708
  // package access
2f2af62dfac7 8010319: Implementation of JEP 181: Nest-Based Access Control
dholmes
parents: 50304
diff changeset
   709
  if (!access.is_private() && is_same_class_package(current_class, member_class)) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   710
    return true;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   711
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   712
50735
2f2af62dfac7 8010319: Implementation of JEP 181: Nest-Based Access Control
dholmes
parents: 50304
diff changeset
   713
  // private access between different classes needs a nestmate check, but
51444
3e5d28e6de32 8209301: JVM rename is_anonymous, host_klass to unsafe specific terminology ahead of Unsafe.defineAnonymousClass deprecation
lfoltan
parents: 50735
diff changeset
   714
  // not for unsafe anonymous classes - so check host_class
50735
2f2af62dfac7 8010319: Implementation of JEP 181: Nest-Based Access Control
dholmes
parents: 50304
diff changeset
   715
  if (access.is_private() && host_class == current_class) {
2f2af62dfac7 8010319: Implementation of JEP 181: Nest-Based Access Control
dholmes
parents: 50304
diff changeset
   716
    if (current_class->is_instance_klass() && member_class->is_instance_klass() ) {
2f2af62dfac7 8010319: Implementation of JEP 181: Nest-Based Access Control
dholmes
parents: 50304
diff changeset
   717
      InstanceKlass* cur_ik = const_cast<InstanceKlass*>(InstanceKlass::cast(current_class));
2f2af62dfac7 8010319: Implementation of JEP 181: Nest-Based Access Control
dholmes
parents: 50304
diff changeset
   718
      InstanceKlass* field_ik = const_cast<InstanceKlass*>(InstanceKlass::cast(member_class));
2f2af62dfac7 8010319: Implementation of JEP 181: Nest-Based Access Control
dholmes
parents: 50304
diff changeset
   719
      // Nestmate access checks may require resolution and validation of the nest-host.
2f2af62dfac7 8010319: Implementation of JEP 181: Nest-Based Access Control
dholmes
parents: 50304
diff changeset
   720
      // It is up to the caller to check for pending exceptions and handle appropriately.
2f2af62dfac7 8010319: Implementation of JEP 181: Nest-Based Access Control
dholmes
parents: 50304
diff changeset
   721
      bool access = cur_ik->has_nestmate_access_to(field_ik, CHECK_false);
2f2af62dfac7 8010319: Implementation of JEP 181: Nest-Based Access Control
dholmes
parents: 50304
diff changeset
   722
      if (access) {
2f2af62dfac7 8010319: Implementation of JEP 181: Nest-Based Access Control
dholmes
parents: 50304
diff changeset
   723
        guarantee(resolved_class->is_subclass_of(member_class), "must be!");
2f2af62dfac7 8010319: Implementation of JEP 181: Nest-Based Access Control
dholmes
parents: 50304
diff changeset
   724
        return true;
2f2af62dfac7 8010319: Implementation of JEP 181: Nest-Based Access Control
dholmes
parents: 50304
diff changeset
   725
      }
2f2af62dfac7 8010319: Implementation of JEP 181: Nest-Based Access Control
dholmes
parents: 50304
diff changeset
   726
    }
2f2af62dfac7 8010319: Implementation of JEP 181: Nest-Based Access Control
dholmes
parents: 50304
diff changeset
   727
  }
2f2af62dfac7 8010319: Implementation of JEP 181: Nest-Based Access Control
dholmes
parents: 50304
diff changeset
   728
37301
a936b4e01afb 8137058: Clear out all non-Critical APIs from sun.reflect
chegar
parents: 37179
diff changeset
   729
  // Allow all accesses from jdk/internal/reflect/MagicAccessorImpl subclasses to
25057
f38210f84f8c 8031819: Remove legacy jdk checks and code
hseigel
parents: 24456
diff changeset
   730
  // succeed trivially.
f38210f84f8c 8031819: Remove legacy jdk checks and code
hseigel
parents: 24456
diff changeset
   731
  if (current_class->is_subclass_of(SystemDictionary::reflect_MagicAccessorImpl_klass())) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   732
    return true;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   733
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   734
50735
2f2af62dfac7 8010319: Implementation of JEP 181: Nest-Based Access Control
dholmes
parents: 50304
diff changeset
   735
  // Check for special relaxations
2f2af62dfac7 8010319: Implementation of JEP 181: Nest-Based Access Control
dholmes
parents: 50304
diff changeset
   736
  return can_relax_access_check_for(current_class, member_class, classloader_only);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   737
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   738
34666
1c7168ea0034 8140485: Class load and creation cleanup
mgronlun
parents: 34238
diff changeset
   739
bool Reflection::is_same_class_package(const Klass* class1, const Klass* class2) {
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 13391
diff changeset
   740
  return InstanceKlass::cast(class1)->is_same_class_package(class2);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   741
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   742
489c9b5090e2 Initial load
duke
parents:
diff changeset
   743
// Checks that the 'outer' klass has declared 'inner' as being an inner klass. If not,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   744
// throw an incompatible class change exception
2332
5c7b6f4ce0a1 6814659: separable cleanups and subroutines for 6655638
jrose
parents: 1550
diff changeset
   745
// If inner_is_member, require the inner to be a member of the outer.
51444
3e5d28e6de32 8209301: JVM rename is_anonymous, host_klass to unsafe specific terminology ahead of Unsafe.defineAnonymousClass deprecation
lfoltan
parents: 50735
diff changeset
   746
// If !inner_is_member, require the inner to be unsafe anonymous (a non-member).
2332
5c7b6f4ce0a1 6814659: separable cleanups and subroutines for 6655638
jrose
parents: 1550
diff changeset
   747
// Caller is responsible for figuring out in advance which case must be true.
46329
53ccc37bda19 8155672: Remove instanceKlassHandles and KlassHandles
coleenp
parents: 46289
diff changeset
   748
void Reflection::check_for_inner_class(const InstanceKlass* outer, const InstanceKlass* inner,
2332
5c7b6f4ce0a1 6814659: separable cleanups and subroutines for 6655638
jrose
parents: 1550
diff changeset
   749
                                       bool inner_is_member, TRAPS) {
12231
6a9cfc59a18a 7109878: The instanceKlass EnclosingMethhod attribute fields can be folded into the _inner_class field.
jiangli
parents: 10504
diff changeset
   750
  InnerClassesIterator iter(outer);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   751
  constantPoolHandle cp   (THREAD, outer->constants());
12231
6a9cfc59a18a 7109878: The instanceKlass EnclosingMethhod attribute fields can be folded into the _inner_class field.
jiangli
parents: 10504
diff changeset
   752
  for (; !iter.done(); iter.next()) {
6a9cfc59a18a 7109878: The instanceKlass EnclosingMethhod attribute fields can be folded into the _inner_class field.
jiangli
parents: 10504
diff changeset
   753
     int ioff = iter.inner_class_info_index();
6a9cfc59a18a 7109878: The instanceKlass EnclosingMethhod attribute fields can be folded into the _inner_class field.
jiangli
parents: 10504
diff changeset
   754
     int ooff = iter.outer_class_info_index();
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   755
2332
5c7b6f4ce0a1 6814659: separable cleanups and subroutines for 6655638
jrose
parents: 1550
diff changeset
   756
     if (inner_is_member && ioff != 0 && ooff != 0) {
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 13391
diff changeset
   757
        Klass* o = cp->klass_at(ooff, CHECK);
46329
53ccc37bda19 8155672: Remove instanceKlassHandles and KlassHandles
coleenp
parents: 46289
diff changeset
   758
        if (o == outer) {
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 13391
diff changeset
   759
          Klass* i = cp->klass_at(ioff, CHECK);
46329
53ccc37bda19 8155672: Remove instanceKlassHandles and KlassHandles
coleenp
parents: 46289
diff changeset
   760
          if (i == inner) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   761
            return;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   762
          }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   763
        }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   764
     }
2332
5c7b6f4ce0a1 6814659: separable cleanups and subroutines for 6655638
jrose
parents: 1550
diff changeset
   765
     if (!inner_is_member && ioff != 0 && ooff == 0 &&
5c7b6f4ce0a1 6814659: separable cleanups and subroutines for 6655638
jrose
parents: 1550
diff changeset
   766
         cp->klass_name_at_matches(inner, ioff)) {
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 13391
diff changeset
   767
        Klass* i = cp->klass_at(ioff, CHECK);
46329
53ccc37bda19 8155672: Remove instanceKlassHandles and KlassHandles
coleenp
parents: 46289
diff changeset
   768
        if (i == inner) {
2332
5c7b6f4ce0a1 6814659: separable cleanups and subroutines for 6655638
jrose
parents: 1550
diff changeset
   769
          return;
5c7b6f4ce0a1 6814659: separable cleanups and subroutines for 6655638
jrose
parents: 1550
diff changeset
   770
        }
5c7b6f4ce0a1 6814659: separable cleanups and subroutines for 6655638
jrose
parents: 1550
diff changeset
   771
     }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   772
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   773
489c9b5090e2 Initial load
duke
parents:
diff changeset
   774
  // 'inner' not declared as an inner klass in outer
489c9b5090e2 Initial load
duke
parents:
diff changeset
   775
  ResourceMark rm(THREAD);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   776
  Exceptions::fthrow(
489c9b5090e2 Initial load
duke
parents:
diff changeset
   777
    THREAD_AND_LOCATION,
8076
96d498ec7ae1 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 7397
diff changeset
   778
    vmSymbols::java_lang_IncompatibleClassChangeError(),
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   779
    "%s and %s disagree on InnerClasses attribute",
489c9b5090e2 Initial load
duke
parents:
diff changeset
   780
    outer->external_name(),
489c9b5090e2 Initial load
duke
parents:
diff changeset
   781
    inner->external_name()
489c9b5090e2 Initial load
duke
parents:
diff changeset
   782
  );
489c9b5090e2 Initial load
duke
parents:
diff changeset
   783
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   784
489c9b5090e2 Initial load
duke
parents:
diff changeset
   785
// Utility method converting a single SignatureStream element into java.lang.Class instance
46727
6e4a84748e2c 8183039: Re-examine methodHandle methods uninlined by 8144256
coleenp
parents: 46630
diff changeset
   786
static oop get_mirror_from_signature(const methodHandle& method,
34666
1c7168ea0034 8140485: Class load and creation cleanup
mgronlun
parents: 34238
diff changeset
   787
                                     SignatureStream* ss,
1c7168ea0034 8140485: Class load and creation cleanup
mgronlun
parents: 34238
diff changeset
   788
                                     TRAPS) {
48836
423bcbb288ff 8196601: IllegalAccessError: cannot access class jdk.jfr.internal.handlers.EventHandler
lfoltan
parents: 48826
diff changeset
   789
34666
1c7168ea0034 8140485: Class load and creation cleanup
mgronlun
parents: 34238
diff changeset
   790
48836
423bcbb288ff 8196601: IllegalAccessError: cannot access class jdk.jfr.internal.handlers.EventHandler
lfoltan
parents: 48826
diff changeset
   791
  if (T_OBJECT == ss->type() || T_ARRAY == ss->type()) {
423bcbb288ff 8196601: IllegalAccessError: cannot access class jdk.jfr.internal.handlers.EventHandler
lfoltan
parents: 48826
diff changeset
   792
    Symbol* name = ss->as_symbol(CHECK_NULL);
423bcbb288ff 8196601: IllegalAccessError: cannot access class jdk.jfr.internal.handlers.EventHandler
lfoltan
parents: 48826
diff changeset
   793
    oop loader = method->method_holder()->class_loader();
423bcbb288ff 8196601: IllegalAccessError: cannot access class jdk.jfr.internal.handlers.EventHandler
lfoltan
parents: 48826
diff changeset
   794
    oop protection_domain = method->method_holder()->protection_domain();
423bcbb288ff 8196601: IllegalAccessError: cannot access class jdk.jfr.internal.handlers.EventHandler
lfoltan
parents: 48826
diff changeset
   795
    const Klass* k = SystemDictionary::resolve_or_fail(name,
423bcbb288ff 8196601: IllegalAccessError: cannot access class jdk.jfr.internal.handlers.EventHandler
lfoltan
parents: 48826
diff changeset
   796
                                                       Handle(THREAD, loader),
423bcbb288ff 8196601: IllegalAccessError: cannot access class jdk.jfr.internal.handlers.EventHandler
lfoltan
parents: 48826
diff changeset
   797
                                                       Handle(THREAD, protection_domain),
423bcbb288ff 8196601: IllegalAccessError: cannot access class jdk.jfr.internal.handlers.EventHandler
lfoltan
parents: 48826
diff changeset
   798
                                                       true,
423bcbb288ff 8196601: IllegalAccessError: cannot access class jdk.jfr.internal.handlers.EventHandler
lfoltan
parents: 48826
diff changeset
   799
                                                       CHECK_NULL);
423bcbb288ff 8196601: IllegalAccessError: cannot access class jdk.jfr.internal.handlers.EventHandler
lfoltan
parents: 48826
diff changeset
   800
    if (log_is_enabled(Debug, class, resolve)) {
423bcbb288ff 8196601: IllegalAccessError: cannot access class jdk.jfr.internal.handlers.EventHandler
lfoltan
parents: 48826
diff changeset
   801
      trace_class_resolution(k);
34666
1c7168ea0034 8140485: Class load and creation cleanup
mgronlun
parents: 34238
diff changeset
   802
    }
48836
423bcbb288ff 8196601: IllegalAccessError: cannot access class jdk.jfr.internal.handlers.EventHandler
lfoltan
parents: 48826
diff changeset
   803
    return k->java_mirror();
34666
1c7168ea0034 8140485: Class load and creation cleanup
mgronlun
parents: 34238
diff changeset
   804
  }
1c7168ea0034 8140485: Class load and creation cleanup
mgronlun
parents: 34238
diff changeset
   805
1c7168ea0034 8140485: Class load and creation cleanup
mgronlun
parents: 34238
diff changeset
   806
  assert(ss->type() != T_VOID || ss->at_return_type(),
1c7168ea0034 8140485: Class load and creation cleanup
mgronlun
parents: 34238
diff changeset
   807
    "T_VOID should only appear as return type");
1c7168ea0034 8140485: Class load and creation cleanup
mgronlun
parents: 34238
diff changeset
   808
48836
423bcbb288ff 8196601: IllegalAccessError: cannot access class jdk.jfr.internal.handlers.EventHandler
lfoltan
parents: 48826
diff changeset
   809
  return java_lang_Class::primitive_mirror(ss->type());
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   810
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   811
46727
6e4a84748e2c 8183039: Re-examine methodHandle methods uninlined by 8144256
coleenp
parents: 46630
diff changeset
   812
static objArrayHandle get_parameter_types(const methodHandle& method,
34666
1c7168ea0034 8140485: Class load and creation cleanup
mgronlun
parents: 34238
diff changeset
   813
                                          int parameter_count,
1c7168ea0034 8140485: Class load and creation cleanup
mgronlun
parents: 34238
diff changeset
   814
                                          oop* return_type,
1c7168ea0034 8140485: Class load and creation cleanup
mgronlun
parents: 34238
diff changeset
   815
                                          TRAPS) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   816
  // Allocate array holding parameter types (java.lang.Class instances)
4571
80b553bddc26 6914300: ciEnv should export all well known classes
never
parents: 2332
diff changeset
   817
  objArrayOop m = oopFactory::new_objArray(SystemDictionary::Class_klass(), parameter_count, CHECK_(objArrayHandle()));
34666
1c7168ea0034 8140485: Class load and creation cleanup
mgronlun
parents: 34238
diff changeset
   818
  objArrayHandle mirrors(THREAD, m);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   819
  int index = 0;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   820
  // Collect parameter types
8076
96d498ec7ae1 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 7397
diff changeset
   821
  ResourceMark rm(THREAD);
34666
1c7168ea0034 8140485: Class load and creation cleanup
mgronlun
parents: 34238
diff changeset
   822
  Symbol*  signature = method->signature();
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   823
  SignatureStream ss(signature);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   824
  while (!ss.at_return_type()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   825
    oop mirror = get_mirror_from_signature(method, &ss, CHECK_(objArrayHandle()));
489c9b5090e2 Initial load
duke
parents:
diff changeset
   826
    mirrors->obj_at_put(index++, mirror);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   827
    ss.next();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   828
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   829
  assert(index == parameter_count, "invalid parameter count");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   830
  if (return_type != NULL) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   831
    // Collect return type as well
489c9b5090e2 Initial load
duke
parents:
diff changeset
   832
    assert(ss.at_return_type(), "return type should be present");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   833
    *return_type = get_mirror_from_signature(method, &ss, CHECK_(objArrayHandle()));
489c9b5090e2 Initial load
duke
parents:
diff changeset
   834
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   835
  return mirrors;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   836
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   837
46727
6e4a84748e2c 8183039: Re-examine methodHandle methods uninlined by 8144256
coleenp
parents: 46630
diff changeset
   838
static objArrayHandle get_exception_types(const methodHandle& method, TRAPS) {
27680
8ecc0871c18e 8064811: Use THREAD instead of CHECK_NULL in return statements
stefank
parents: 27677
diff changeset
   839
  return method->resolved_checked_exceptions(THREAD);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   840
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   841
46329
53ccc37bda19 8155672: Remove instanceKlassHandles and KlassHandles
coleenp
parents: 46289
diff changeset
   842
static Handle new_type(Symbol* signature, Klass* k, TRAPS) {
48836
423bcbb288ff 8196601: IllegalAccessError: cannot access class jdk.jfr.internal.handlers.EventHandler
lfoltan
parents: 48826
diff changeset
   843
  // Basic types
423bcbb288ff 8196601: IllegalAccessError: cannot access class jdk.jfr.internal.handlers.EventHandler
lfoltan
parents: 48826
diff changeset
   844
  BasicType type = vmSymbols::signature_type(signature);
423bcbb288ff 8196601: IllegalAccessError: cannot access class jdk.jfr.internal.handlers.EventHandler
lfoltan
parents: 48826
diff changeset
   845
  if (type != T_OBJECT) {
423bcbb288ff 8196601: IllegalAccessError: cannot access class jdk.jfr.internal.handlers.EventHandler
lfoltan
parents: 48826
diff changeset
   846
    return Handle(THREAD, Universe::java_mirror(type));
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   847
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   848
48836
423bcbb288ff 8196601: IllegalAccessError: cannot access class jdk.jfr.internal.handlers.EventHandler
lfoltan
parents: 48826
diff changeset
   849
  Klass* result =
423bcbb288ff 8196601: IllegalAccessError: cannot access class jdk.jfr.internal.handlers.EventHandler
lfoltan
parents: 48826
diff changeset
   850
    SystemDictionary::resolve_or_fail(signature,
423bcbb288ff 8196601: IllegalAccessError: cannot access class jdk.jfr.internal.handlers.EventHandler
lfoltan
parents: 48826
diff changeset
   851
                                      Handle(THREAD, k->class_loader()),
423bcbb288ff 8196601: IllegalAccessError: cannot access class jdk.jfr.internal.handlers.EventHandler
lfoltan
parents: 48826
diff changeset
   852
                                      Handle(THREAD, k->protection_domain()),
423bcbb288ff 8196601: IllegalAccessError: cannot access class jdk.jfr.internal.handlers.EventHandler
lfoltan
parents: 48826
diff changeset
   853
                                      true, CHECK_(Handle()));
423bcbb288ff 8196601: IllegalAccessError: cannot access class jdk.jfr.internal.handlers.EventHandler
lfoltan
parents: 48826
diff changeset
   854
423bcbb288ff 8196601: IllegalAccessError: cannot access class jdk.jfr.internal.handlers.EventHandler
lfoltan
parents: 48826
diff changeset
   855
  if (log_is_enabled(Debug, class, resolve)) {
423bcbb288ff 8196601: IllegalAccessError: cannot access class jdk.jfr.internal.handlers.EventHandler
lfoltan
parents: 48826
diff changeset
   856
    trace_class_resolution(result);
423bcbb288ff 8196601: IllegalAccessError: cannot access class jdk.jfr.internal.handlers.EventHandler
lfoltan
parents: 48826
diff changeset
   857
  }
423bcbb288ff 8196601: IllegalAccessError: cannot access class jdk.jfr.internal.handlers.EventHandler
lfoltan
parents: 48826
diff changeset
   858
423bcbb288ff 8196601: IllegalAccessError: cannot access class jdk.jfr.internal.handlers.EventHandler
lfoltan
parents: 48826
diff changeset
   859
  oop nt = result->java_mirror();
423bcbb288ff 8196601: IllegalAccessError: cannot access class jdk.jfr.internal.handlers.EventHandler
lfoltan
parents: 48826
diff changeset
   860
  return Handle(THREAD, nt);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   861
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   862
489c9b5090e2 Initial load
duke
parents:
diff changeset
   863
33593
60764a78fa5c 8140274: methodHandles and constantPoolHandles should be passed as const references
coleenp
parents: 31615
diff changeset
   864
oop Reflection::new_method(const methodHandle& method, bool for_constant_pool_access, TRAPS) {
25057
f38210f84f8c 8031819: Remove legacy jdk checks and code
hseigel
parents: 24456
diff changeset
   865
  // Allow sun.reflect.ConstantPool to refer to <clinit> methods as java.lang.reflect.Methods.
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   866
  assert(!method()->is_initializer() ||
25057
f38210f84f8c 8031819: Remove legacy jdk checks and code
hseigel
parents: 24456
diff changeset
   867
         (for_constant_pool_access && method()->is_static()),
f38210f84f8c 8031819: Remove legacy jdk checks and code
hseigel
parents: 24456
diff changeset
   868
         "should call new_constructor instead");
46329
53ccc37bda19 8155672: Remove instanceKlassHandles and KlassHandles
coleenp
parents: 46289
diff changeset
   869
  InstanceKlass* holder = method->method_holder();
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   870
  int slot = method->method_idnum();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   871
8076
96d498ec7ae1 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 7397
diff changeset
   872
  Symbol*  signature  = method->signature();
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   873
  int parameter_count = ArgumentCount(signature).size();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   874
  oop return_type_oop = NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   875
  objArrayHandle parameter_types = get_parameter_types(method, parameter_count, &return_type_oop, CHECK_NULL);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   876
  if (parameter_types.is_null() || return_type_oop == NULL) return NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   877
489c9b5090e2 Initial load
duke
parents:
diff changeset
   878
  Handle return_type(THREAD, return_type_oop);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   879
489c9b5090e2 Initial load
duke
parents:
diff changeset
   880
  objArrayHandle exception_types = get_exception_types(method, CHECK_NULL);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   881
489c9b5090e2 Initial load
duke
parents:
diff changeset
   882
  if (exception_types.is_null()) return NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   883
8076
96d498ec7ae1 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 7397
diff changeset
   884
  Symbol*  method_name = method->name();
24456
8c7933fa5a1f 8025580: Temporary flags: UseNewReflection and ReflectionWrapResolutionErrors
coleenp
parents: 24426
diff changeset
   885
  oop name_oop = StringTable::intern(method_name, CHECK_NULL);
8c7933fa5a1f 8025580: Temporary flags: UseNewReflection and ReflectionWrapResolutionErrors
coleenp
parents: 24426
diff changeset
   886
  Handle name = Handle(THREAD, name_oop);
8076
96d498ec7ae1 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 7397
diff changeset
   887
  if (name == NULL) return NULL;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   888
34666
1c7168ea0034 8140485: Class load and creation cleanup
mgronlun
parents: 34238
diff changeset
   889
  const int modifiers = method->access_flags().as_int() & JVM_RECOGNIZED_METHOD_MODIFIERS;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   890
489c9b5090e2 Initial load
duke
parents:
diff changeset
   891
  Handle mh = java_lang_reflect_Method::create(CHECK_NULL);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   892
489c9b5090e2 Initial load
duke
parents:
diff changeset
   893
  java_lang_reflect_Method::set_clazz(mh(), holder->java_mirror());
489c9b5090e2 Initial load
duke
parents:
diff changeset
   894
  java_lang_reflect_Method::set_slot(mh(), slot);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   895
  java_lang_reflect_Method::set_name(mh(), name());
489c9b5090e2 Initial load
duke
parents:
diff changeset
   896
  java_lang_reflect_Method::set_return_type(mh(), return_type());
489c9b5090e2 Initial load
duke
parents:
diff changeset
   897
  java_lang_reflect_Method::set_parameter_types(mh(), parameter_types());
489c9b5090e2 Initial load
duke
parents:
diff changeset
   898
  java_lang_reflect_Method::set_exception_types(mh(), exception_types());
489c9b5090e2 Initial load
duke
parents:
diff changeset
   899
  java_lang_reflect_Method::set_modifiers(mh(), modifiers);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   900
  java_lang_reflect_Method::set_override(mh(), false);
51807
3221f5e14866 8198717: Remove compute_optional_offset
coleenp
parents: 51467
diff changeset
   901
  if (method->generic_signature() != NULL) {
8076
96d498ec7ae1 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 7397
diff changeset
   902
    Symbol*  gs = method->generic_signature();
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   903
    Handle sig = java_lang_String::create_from_symbol(gs, CHECK_NULL);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   904
    java_lang_reflect_Method::set_signature(mh(), sig());
489c9b5090e2 Initial load
duke
parents:
diff changeset
   905
  }
51807
3221f5e14866 8198717: Remove compute_optional_offset
coleenp
parents: 51467
diff changeset
   906
  typeArrayOop an_oop = Annotations::make_java_array(method->annotations(), CHECK_NULL);
3221f5e14866 8198717: Remove compute_optional_offset
coleenp
parents: 51467
diff changeset
   907
  java_lang_reflect_Method::set_annotations(mh(), an_oop);
3221f5e14866 8198717: Remove compute_optional_offset
coleenp
parents: 51467
diff changeset
   908
  an_oop = Annotations::make_java_array(method->parameter_annotations(), CHECK_NULL);
3221f5e14866 8198717: Remove compute_optional_offset
coleenp
parents: 51467
diff changeset
   909
  java_lang_reflect_Method::set_parameter_annotations(mh(), an_oop);
3221f5e14866 8198717: Remove compute_optional_offset
coleenp
parents: 51467
diff changeset
   910
  an_oop = Annotations::make_java_array(method->annotation_default(), CHECK_NULL);
3221f5e14866 8198717: Remove compute_optional_offset
coleenp
parents: 51467
diff changeset
   911
  java_lang_reflect_Method::set_annotation_default(mh(), an_oop);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   912
  return mh();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   913
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   914
489c9b5090e2 Initial load
duke
parents:
diff changeset
   915
33593
60764a78fa5c 8140274: methodHandles and constantPoolHandles should be passed as const references
coleenp
parents: 31615
diff changeset
   916
oop Reflection::new_constructor(const methodHandle& method, TRAPS) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   917
  assert(method()->is_initializer(), "should call new_method instead");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   918
46329
53ccc37bda19 8155672: Remove instanceKlassHandles and KlassHandles
coleenp
parents: 46289
diff changeset
   919
  InstanceKlass* holder = method->method_holder();
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   920
  int slot = method->method_idnum();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   921
8076
96d498ec7ae1 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 7397
diff changeset
   922
  Symbol*  signature  = method->signature();
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   923
  int parameter_count = ArgumentCount(signature).size();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   924
  objArrayHandle parameter_types = get_parameter_types(method, parameter_count, NULL, CHECK_NULL);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   925
  if (parameter_types.is_null()) return NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   926
489c9b5090e2 Initial load
duke
parents:
diff changeset
   927
  objArrayHandle exception_types = get_exception_types(method, CHECK_NULL);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   928
  if (exception_types.is_null()) return NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   929
34666
1c7168ea0034 8140485: Class load and creation cleanup
mgronlun
parents: 34238
diff changeset
   930
  const int modifiers = method->access_flags().as_int() & JVM_RECOGNIZED_METHOD_MODIFIERS;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   931
489c9b5090e2 Initial load
duke
parents:
diff changeset
   932
  Handle ch = java_lang_reflect_Constructor::create(CHECK_NULL);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   933
489c9b5090e2 Initial load
duke
parents:
diff changeset
   934
  java_lang_reflect_Constructor::set_clazz(ch(), holder->java_mirror());
489c9b5090e2 Initial load
duke
parents:
diff changeset
   935
  java_lang_reflect_Constructor::set_slot(ch(), slot);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   936
  java_lang_reflect_Constructor::set_parameter_types(ch(), parameter_types());
489c9b5090e2 Initial load
duke
parents:
diff changeset
   937
  java_lang_reflect_Constructor::set_exception_types(ch(), exception_types());
489c9b5090e2 Initial load
duke
parents:
diff changeset
   938
  java_lang_reflect_Constructor::set_modifiers(ch(), modifiers);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   939
  java_lang_reflect_Constructor::set_override(ch(), false);
51807
3221f5e14866 8198717: Remove compute_optional_offset
coleenp
parents: 51467
diff changeset
   940
  if (method->generic_signature() != NULL) {
8076
96d498ec7ae1 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 7397
diff changeset
   941
    Symbol*  gs = method->generic_signature();
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   942
    Handle sig = java_lang_String::create_from_symbol(gs, CHECK_NULL);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   943
    java_lang_reflect_Constructor::set_signature(ch(), sig());
489c9b5090e2 Initial load
duke
parents:
diff changeset
   944
  }
51807
3221f5e14866 8198717: Remove compute_optional_offset
coleenp
parents: 51467
diff changeset
   945
  typeArrayOop an_oop = Annotations::make_java_array(method->annotations(), CHECK_NULL);
3221f5e14866 8198717: Remove compute_optional_offset
coleenp
parents: 51467
diff changeset
   946
  java_lang_reflect_Constructor::set_annotations(ch(), an_oop);
3221f5e14866 8198717: Remove compute_optional_offset
coleenp
parents: 51467
diff changeset
   947
  an_oop = Annotations::make_java_array(method->parameter_annotations(), CHECK_NULL);
3221f5e14866 8198717: Remove compute_optional_offset
coleenp
parents: 51467
diff changeset
   948
  java_lang_reflect_Constructor::set_parameter_annotations(ch(), an_oop);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   949
  return ch();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   950
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   951
489c9b5090e2 Initial load
duke
parents:
diff changeset
   952
24456
8c7933fa5a1f 8025580: Temporary flags: UseNewReflection and ReflectionWrapResolutionErrors
coleenp
parents: 24426
diff changeset
   953
oop Reflection::new_field(fieldDescriptor* fd, TRAPS) {
8076
96d498ec7ae1 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 7397
diff changeset
   954
  Symbol*  field_name = fd->name();
24456
8c7933fa5a1f 8025580: Temporary flags: UseNewReflection and ReflectionWrapResolutionErrors
coleenp
parents: 24426
diff changeset
   955
  oop name_oop = StringTable::intern(field_name, CHECK_NULL);
8c7933fa5a1f 8025580: Temporary flags: UseNewReflection and ReflectionWrapResolutionErrors
coleenp
parents: 24426
diff changeset
   956
  Handle name = Handle(THREAD, name_oop);
8076
96d498ec7ae1 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 7397
diff changeset
   957
  Symbol*  signature  = fd->signature();
46329
53ccc37bda19 8155672: Remove instanceKlassHandles and KlassHandles
coleenp
parents: 46289
diff changeset
   958
  InstanceKlass* holder = fd->field_holder();
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   959
  Handle type = new_type(signature, holder, CHECK_NULL);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   960
  Handle rh  = java_lang_reflect_Field::create(CHECK_NULL);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   961
14391
df0a1573d5bd 8000725: NPG: method_holder() and pool_holder() and pool_holder field should be InstanceKlass
coleenp
parents: 14385
diff changeset
   962
  java_lang_reflect_Field::set_clazz(rh(), fd->field_holder()->java_mirror());
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   963
  java_lang_reflect_Field::set_slot(rh(), fd->index());
489c9b5090e2 Initial load
duke
parents:
diff changeset
   964
  java_lang_reflect_Field::set_name(rh(), name());
489c9b5090e2 Initial load
duke
parents:
diff changeset
   965
  java_lang_reflect_Field::set_type(rh(), type());
489c9b5090e2 Initial load
duke
parents:
diff changeset
   966
  // Note the ACC_ANNOTATION bit, which is a per-class access flag, is never set here.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   967
  java_lang_reflect_Field::set_modifiers(rh(), fd->access_flags().as_int() & JVM_RECOGNIZED_FIELD_MODIFIERS);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   968
  java_lang_reflect_Field::set_override(rh(), false);
51807
3221f5e14866 8198717: Remove compute_optional_offset
coleenp
parents: 51467
diff changeset
   969
  if (fd->has_generic_signature()) {
8076
96d498ec7ae1 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 7397
diff changeset
   970
    Symbol*  gs = fd->generic_signature();
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   971
    Handle sig = java_lang_String::create_from_symbol(gs, CHECK_NULL);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   972
    java_lang_reflect_Field::set_signature(rh(), sig());
489c9b5090e2 Initial load
duke
parents:
diff changeset
   973
  }
51807
3221f5e14866 8198717: Remove compute_optional_offset
coleenp
parents: 51467
diff changeset
   974
  typeArrayOop an_oop = Annotations::make_java_array(fd->annotations(), CHECK_NULL);
3221f5e14866 8198717: Remove compute_optional_offset
coleenp
parents: 51467
diff changeset
   975
  java_lang_reflect_Field::set_annotations(rh(), an_oop);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   976
  return rh();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   977
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   978
15102
0a86564e5f61 8004728: Add hotspot support for parameter reflection
coleenp
parents: 15097
diff changeset
   979
oop Reflection::new_parameter(Handle method, int index, Symbol* sym,
0a86564e5f61 8004728: Add hotspot support for parameter reflection
coleenp
parents: 15097
diff changeset
   980
                              int flags, TRAPS) {
15194
a35093d73168 8006005: Fix constant pool index validation and alignment trap for method parameter reflection
coleenp
parents: 15102
diff changeset
   981
27613
f4773c0d8717 8058322: Zero name_index item of MethodParameters attribute cause MalformedParameterException
emc
parents: 27022
diff changeset
   982
  Handle rh = java_lang_reflect_Parameter::create(CHECK_NULL);
f4773c0d8717 8058322: Zero name_index item of MethodParameters attribute cause MalformedParameterException
emc
parents: 27022
diff changeset
   983
15194
a35093d73168 8006005: Fix constant pool index validation and alignment trap for method parameter reflection
coleenp
parents: 15102
diff changeset
   984
  if(NULL != sym) {
27613
f4773c0d8717 8058322: Zero name_index item of MethodParameters attribute cause MalformedParameterException
emc
parents: 27022
diff changeset
   985
    Handle name = java_lang_String::create_from_symbol(sym, CHECK_NULL);
f4773c0d8717 8058322: Zero name_index item of MethodParameters attribute cause MalformedParameterException
emc
parents: 27022
diff changeset
   986
    java_lang_reflect_Parameter::set_name(rh(), name());
15194
a35093d73168 8006005: Fix constant pool index validation and alignment trap for method parameter reflection
coleenp
parents: 15102
diff changeset
   987
  } else {
27613
f4773c0d8717 8058322: Zero name_index item of MethodParameters attribute cause MalformedParameterException
emc
parents: 27022
diff changeset
   988
    java_lang_reflect_Parameter::set_name(rh(), NULL);
15194
a35093d73168 8006005: Fix constant pool index validation and alignment trap for method parameter reflection
coleenp
parents: 15102
diff changeset
   989
  }
a35093d73168 8006005: Fix constant pool index validation and alignment trap for method parameter reflection
coleenp
parents: 15102
diff changeset
   990
15102
0a86564e5f61 8004728: Add hotspot support for parameter reflection
coleenp
parents: 15097
diff changeset
   991
  java_lang_reflect_Parameter::set_modifiers(rh(), flags);
0a86564e5f61 8004728: Add hotspot support for parameter reflection
coleenp
parents: 15097
diff changeset
   992
  java_lang_reflect_Parameter::set_executable(rh(), method());
0a86564e5f61 8004728: Add hotspot support for parameter reflection
coleenp
parents: 15097
diff changeset
   993
  java_lang_reflect_Parameter::set_index(rh(), index);
0a86564e5f61 8004728: Add hotspot support for parameter reflection
coleenp
parents: 15097
diff changeset
   994
  return rh();
0a86564e5f61 8004728: Add hotspot support for parameter reflection
coleenp
parents: 15097
diff changeset
   995
}
0a86564e5f61 8004728: Add hotspot support for parameter reflection
coleenp
parents: 15097
diff changeset
   996
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   997
46329
53ccc37bda19 8155672: Remove instanceKlassHandles and KlassHandles
coleenp
parents: 46289
diff changeset
   998
static methodHandle resolve_interface_call(InstanceKlass* klass,
34666
1c7168ea0034 8140485: Class load and creation cleanup
mgronlun
parents: 34238
diff changeset
   999
                                           const methodHandle& method,
46329
53ccc37bda19 8155672: Remove instanceKlassHandles and KlassHandles
coleenp
parents: 46289
diff changeset
  1000
                                           Klass* recv_klass,
34666
1c7168ea0034 8140485: Class load and creation cleanup
mgronlun
parents: 34238
diff changeset
  1001
                                           Handle receiver,
1c7168ea0034 8140485: Class load and creation cleanup
mgronlun
parents: 34238
diff changeset
  1002
                                           TRAPS) {
1c7168ea0034 8140485: Class load and creation cleanup
mgronlun
parents: 34238
diff changeset
  1003
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1004
  assert(!method.is_null() , "method should not be null");
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1005
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1006
  CallInfo info;
8076
96d498ec7ae1 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 7397
diff changeset
  1007
  Symbol*  signature  = method->signature();
96d498ec7ae1 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 7397
diff changeset
  1008
  Symbol*  name       = method->name();
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 29081
diff changeset
  1009
  LinkResolver::resolve_interface_call(info, receiver, recv_klass,
38719
133bf85c3f36 8145148: InterfaceMethod CP entry pointing to a class should cause ICCE
coleenp
parents: 38151
diff changeset
  1010
                                       LinkInfo(klass, name, signature),
31019
d05fcdd70109 8029567: Clean up linkResolver code
coleenp
parents: 29081
diff changeset
  1011
                                       true,
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1012
                                       CHECK_(methodHandle()));
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1013
  return info.selected_method();
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1014
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1015
34666
1c7168ea0034 8140485: Class load and creation cleanup
mgronlun
parents: 34238
diff changeset
  1016
// Conversion
1c7168ea0034 8140485: Class load and creation cleanup
mgronlun
parents: 34238
diff changeset
  1017
static BasicType basic_type_mirror_to_basic_type(oop basic_type_mirror, TRAPS) {
1c7168ea0034 8140485: Class load and creation cleanup
mgronlun
parents: 34238
diff changeset
  1018
  assert(java_lang_Class::is_primitive(basic_type_mirror),
1c7168ea0034 8140485: Class load and creation cleanup
mgronlun
parents: 34238
diff changeset
  1019
    "just checking");
1c7168ea0034 8140485: Class load and creation cleanup
mgronlun
parents: 34238
diff changeset
  1020
  return java_lang_Class::primitive_type(basic_type_mirror);
1c7168ea0034 8140485: Class load and creation cleanup
mgronlun
parents: 34238
diff changeset
  1021
}
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1022
34666
1c7168ea0034 8140485: Class load and creation cleanup
mgronlun
parents: 34238
diff changeset
  1023
// Narrowing of basic types. Used to create correct jvalues for
1c7168ea0034 8140485: Class load and creation cleanup
mgronlun
parents: 34238
diff changeset
  1024
// boolean, byte, char and short return return values from interpreter
1c7168ea0034 8140485: Class load and creation cleanup
mgronlun
parents: 34238
diff changeset
  1025
// which are returned as ints. Throws IllegalArgumentException.
1c7168ea0034 8140485: Class load and creation cleanup
mgronlun
parents: 34238
diff changeset
  1026
static void narrow(jvalue* value, BasicType narrow_type, TRAPS) {
1c7168ea0034 8140485: Class load and creation cleanup
mgronlun
parents: 34238
diff changeset
  1027
  switch (narrow_type) {
1c7168ea0034 8140485: Class load and creation cleanup
mgronlun
parents: 34238
diff changeset
  1028
  case T_BOOLEAN:
37480
291ee208fb72 8132051: Better byte behavior
coleenp
parents: 37301
diff changeset
  1029
    value->z = (jboolean) (value->i & 1);
34666
1c7168ea0034 8140485: Class load and creation cleanup
mgronlun
parents: 34238
diff changeset
  1030
    return;
1c7168ea0034 8140485: Class load and creation cleanup
mgronlun
parents: 34238
diff changeset
  1031
  case T_BYTE:
1c7168ea0034 8140485: Class load and creation cleanup
mgronlun
parents: 34238
diff changeset
  1032
    value->b = (jbyte)value->i;
1c7168ea0034 8140485: Class load and creation cleanup
mgronlun
parents: 34238
diff changeset
  1033
    return;
1c7168ea0034 8140485: Class load and creation cleanup
mgronlun
parents: 34238
diff changeset
  1034
  case T_CHAR:
1c7168ea0034 8140485: Class load and creation cleanup
mgronlun
parents: 34238
diff changeset
  1035
    value->c = (jchar)value->i;
1c7168ea0034 8140485: Class load and creation cleanup
mgronlun
parents: 34238
diff changeset
  1036
    return;
1c7168ea0034 8140485: Class load and creation cleanup
mgronlun
parents: 34238
diff changeset
  1037
  case T_SHORT:
1c7168ea0034 8140485: Class load and creation cleanup
mgronlun
parents: 34238
diff changeset
  1038
    value->s = (jshort)value->i;
1c7168ea0034 8140485: Class load and creation cleanup
mgronlun
parents: 34238
diff changeset
  1039
    return;
1c7168ea0034 8140485: Class load and creation cleanup
mgronlun
parents: 34238
diff changeset
  1040
  default:
1c7168ea0034 8140485: Class load and creation cleanup
mgronlun
parents: 34238
diff changeset
  1041
    break; // fail
1c7168ea0034 8140485: Class load and creation cleanup
mgronlun
parents: 34238
diff changeset
  1042
  }
1c7168ea0034 8140485: Class load and creation cleanup
mgronlun
parents: 34238
diff changeset
  1043
  THROW_MSG(vmSymbols::java_lang_IllegalArgumentException(), "argument type mismatch");
1c7168ea0034 8140485: Class load and creation cleanup
mgronlun
parents: 34238
diff changeset
  1044
}
1c7168ea0034 8140485: Class load and creation cleanup
mgronlun
parents: 34238
diff changeset
  1045
1c7168ea0034 8140485: Class load and creation cleanup
mgronlun
parents: 34238
diff changeset
  1046
1c7168ea0034 8140485: Class load and creation cleanup
mgronlun
parents: 34238
diff changeset
  1047
// Method call (shared by invoke_method and invoke_constructor)
46329
53ccc37bda19 8155672: Remove instanceKlassHandles and KlassHandles
coleenp
parents: 46289
diff changeset
  1048
static oop invoke(InstanceKlass* klass,
46727
6e4a84748e2c 8183039: Re-examine methodHandle methods uninlined by 8144256
coleenp
parents: 46630
diff changeset
  1049
                  const methodHandle& reflected_method,
34666
1c7168ea0034 8140485: Class load and creation cleanup
mgronlun
parents: 34238
diff changeset
  1050
                  Handle receiver,
1c7168ea0034 8140485: Class load and creation cleanup
mgronlun
parents: 34238
diff changeset
  1051
                  bool override,
1c7168ea0034 8140485: Class load and creation cleanup
mgronlun
parents: 34238
diff changeset
  1052
                  objArrayHandle ptypes,
1c7168ea0034 8140485: Class load and creation cleanup
mgronlun
parents: 34238
diff changeset
  1053
                  BasicType rtype,
1c7168ea0034 8140485: Class load and creation cleanup
mgronlun
parents: 34238
diff changeset
  1054
                  objArrayHandle args,
1c7168ea0034 8140485: Class load and creation cleanup
mgronlun
parents: 34238
diff changeset
  1055
                  bool is_method_invoke,
1c7168ea0034 8140485: Class load and creation cleanup
mgronlun
parents: 34238
diff changeset
  1056
                  TRAPS) {
1c7168ea0034 8140485: Class load and creation cleanup
mgronlun
parents: 34238
diff changeset
  1057
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1058
  ResourceMark rm(THREAD);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1059
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1060
  methodHandle method;      // actual method to invoke
46329
53ccc37bda19 8155672: Remove instanceKlassHandles and KlassHandles
coleenp
parents: 46289
diff changeset
  1061
  Klass* target_klass;      // target klass, receiver's klass for non-static
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1062
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1063
  // Ensure klass is initialized
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1064
  klass->initialize(CHECK_NULL);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1065
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1066
  bool is_static = reflected_method->is_static();
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1067
  if (is_static) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1068
    // ignore receiver argument
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1069
    method = reflected_method;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1070
    target_klass = klass;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1071
  } else {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1072
    // check for null receiver
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1073
    if (receiver.is_null()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1074
      THROW_0(vmSymbols::java_lang_NullPointerException());
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1075
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1076
    // Check class of receiver against class declaring method
46329
53ccc37bda19 8155672: Remove instanceKlassHandles and KlassHandles
coleenp
parents: 46289
diff changeset
  1077
    if (!receiver->is_a(klass)) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1078
      THROW_MSG_0(vmSymbols::java_lang_IllegalArgumentException(), "object is not an instance of declaring class");
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1079
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1080
    // target klass is receiver's klass
46329
53ccc37bda19 8155672: Remove instanceKlassHandles and KlassHandles
coleenp
parents: 46289
diff changeset
  1081
    target_klass = receiver->klass();
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1082
    // no need to resolve if method is private or <init>
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1083
    if (reflected_method->is_private() || reflected_method->name() == vmSymbols::object_initializer_name()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1084
      method = reflected_method;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1085
    } else {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1086
      // resolve based on the receiver
14391
df0a1573d5bd 8000725: NPG: method_holder() and pool_holder() and pool_holder field should be InstanceKlass
coleenp
parents: 14385
diff changeset
  1087
      if (reflected_method->method_holder()->is_interface()) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1088
        // resolve interface call
24456
8c7933fa5a1f 8025580: Temporary flags: UseNewReflection and ReflectionWrapResolutionErrors
coleenp
parents: 24426
diff changeset
  1089
        //
8c7933fa5a1f 8025580: Temporary flags: UseNewReflection and ReflectionWrapResolutionErrors
coleenp
parents: 24426
diff changeset
  1090
        // Match resolution errors with those thrown due to reflection inlining
8c7933fa5a1f 8025580: Temporary flags: UseNewReflection and ReflectionWrapResolutionErrors
coleenp
parents: 24426
diff changeset
  1091
        // Linktime resolution & IllegalAccessCheck already done by Class.getMethod()
8c7933fa5a1f 8025580: Temporary flags: UseNewReflection and ReflectionWrapResolutionErrors
coleenp
parents: 24426
diff changeset
  1092
        method = resolve_interface_call(klass, reflected_method, target_klass, receiver, THREAD);
8c7933fa5a1f 8025580: Temporary flags: UseNewReflection and ReflectionWrapResolutionErrors
coleenp
parents: 24426
diff changeset
  1093
        if (HAS_PENDING_EXCEPTION) {
34666
1c7168ea0034 8140485: Class load and creation cleanup
mgronlun
parents: 34238
diff changeset
  1094
          // Method resolution threw an exception; wrap it in an InvocationTargetException
24456
8c7933fa5a1f 8025580: Temporary flags: UseNewReflection and ReflectionWrapResolutionErrors
coleenp
parents: 24426
diff changeset
  1095
          oop resolution_exception = PENDING_EXCEPTION;
8c7933fa5a1f 8025580: Temporary flags: UseNewReflection and ReflectionWrapResolutionErrors
coleenp
parents: 24426
diff changeset
  1096
          CLEAR_PENDING_EXCEPTION;
8c7933fa5a1f 8025580: Temporary flags: UseNewReflection and ReflectionWrapResolutionErrors
coleenp
parents: 24426
diff changeset
  1097
          // JVMTI has already reported the pending exception
8c7933fa5a1f 8025580: Temporary flags: UseNewReflection and ReflectionWrapResolutionErrors
coleenp
parents: 24426
diff changeset
  1098
          // JVMTI internal flag reset is needed in order to report InvocationTargetException
8c7933fa5a1f 8025580: Temporary flags: UseNewReflection and ReflectionWrapResolutionErrors
coleenp
parents: 24426
diff changeset
  1099
          if (THREAD->is_Java_thread()) {
34666
1c7168ea0034 8140485: Class load and creation cleanup
mgronlun
parents: 34238
diff changeset
  1100
            JvmtiExport::clear_detected_exception((JavaThread*)THREAD);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1101
          }
24456
8c7933fa5a1f 8025580: Temporary flags: UseNewReflection and ReflectionWrapResolutionErrors
coleenp
parents: 24426
diff changeset
  1102
          JavaCallArguments args(Handle(THREAD, resolution_exception));
8c7933fa5a1f 8025580: Temporary flags: UseNewReflection and ReflectionWrapResolutionErrors
coleenp
parents: 24426
diff changeset
  1103
          THROW_ARG_0(vmSymbols::java_lang_reflect_InvocationTargetException(),
34666
1c7168ea0034 8140485: Class load and creation cleanup
mgronlun
parents: 34238
diff changeset
  1104
                      vmSymbols::throwable_void_signature(),
1c7168ea0034 8140485: Class load and creation cleanup
mgronlun
parents: 34238
diff changeset
  1105
                      &args);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1106
        }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1107
      }  else {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1108
        // if the method can be overridden, we resolve using the vtable index.
20017
81eba62e9048 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 18508
diff changeset
  1109
        assert(!reflected_method->has_itable_index(), "");
81eba62e9048 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 18508
diff changeset
  1110
        int index = reflected_method->vtable_index();
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1111
        method = reflected_method;
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 13391
diff changeset
  1112
        if (index != Method::nonvirtual_vtable_index) {
35900
d64cf9290fc4 8148481: Devirtualize Klass::vtable
mgerdin
parents: 35219
diff changeset
  1113
          method = methodHandle(THREAD, target_klass->method_at_vtable(index));
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1114
        }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1115
        if (!method.is_null()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1116
          // Check for abstract methods as well
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1117
          if (method->is_abstract()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1118
            // new default: 6531596
24456
8c7933fa5a1f 8025580: Temporary flags: UseNewReflection and ReflectionWrapResolutionErrors
coleenp
parents: 24426
diff changeset
  1119
            ResourceMark rm(THREAD);
8c7933fa5a1f 8025580: Temporary flags: UseNewReflection and ReflectionWrapResolutionErrors
coleenp
parents: 24426
diff changeset
  1120
            Handle h_origexception = Exceptions::new_exception(THREAD,
34666
1c7168ea0034 8140485: Class load and creation cleanup
mgronlun
parents: 34238
diff changeset
  1121
              vmSymbols::java_lang_AbstractMethodError(),
46329
53ccc37bda19 8155672: Remove instanceKlassHandles and KlassHandles
coleenp
parents: 46289
diff changeset
  1122
              Method::name_and_sig_as_C_string(target_klass,
34666
1c7168ea0034 8140485: Class load and creation cleanup
mgronlun
parents: 34238
diff changeset
  1123
              method->name(),
1c7168ea0034 8140485: Class load and creation cleanup
mgronlun
parents: 34238
diff changeset
  1124
              method->signature()));
24456
8c7933fa5a1f 8025580: Temporary flags: UseNewReflection and ReflectionWrapResolutionErrors
coleenp
parents: 24426
diff changeset
  1125
            JavaCallArguments args(h_origexception);
8c7933fa5a1f 8025580: Temporary flags: UseNewReflection and ReflectionWrapResolutionErrors
coleenp
parents: 24426
diff changeset
  1126
            THROW_ARG_0(vmSymbols::java_lang_reflect_InvocationTargetException(),
8c7933fa5a1f 8025580: Temporary flags: UseNewReflection and ReflectionWrapResolutionErrors
coleenp
parents: 24426
diff changeset
  1127
              vmSymbols::throwable_void_signature(),
8c7933fa5a1f 8025580: Temporary flags: UseNewReflection and ReflectionWrapResolutionErrors
coleenp
parents: 24426
diff changeset
  1128
              &args);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1129
          }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1130
        }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1131
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1132
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1133
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1134
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1135
  // I believe this is a ShouldNotGetHere case which requires
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1136
  // an internal vtable bug. If you ever get this please let Karen know.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1137
  if (method.is_null()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1138
    ResourceMark rm(THREAD);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1139
    THROW_MSG_0(vmSymbols::java_lang_NoSuchMethodError(),
46329
53ccc37bda19 8155672: Remove instanceKlassHandles and KlassHandles
coleenp
parents: 46289
diff changeset
  1140
                Method::name_and_sig_as_C_string(klass,
34666
1c7168ea0034 8140485: Class load and creation cleanup
mgronlun
parents: 34238
diff changeset
  1141
                reflected_method->name(),
1c7168ea0034 8140485: Class load and creation cleanup
mgronlun
parents: 34238
diff changeset
  1142
                reflected_method->signature()));
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1143
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1144
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1145
  assert(ptypes->is_objArray(), "just checking");
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1146
  int args_len = args.is_null() ? 0 : args->length();
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1147
  // Check number of arguments
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1148
  if (ptypes->length() != args_len) {
34666
1c7168ea0034 8140485: Class load and creation cleanup
mgronlun
parents: 34238
diff changeset
  1149
    THROW_MSG_0(vmSymbols::java_lang_IllegalArgumentException(),
1c7168ea0034 8140485: Class load and creation cleanup
mgronlun
parents: 34238
diff changeset
  1150
                "wrong number of arguments");
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1151
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1152
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1153
  // Create object to contain parameters for the JavaCall
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1154
  JavaCallArguments java_args(method->size_of_parameters());
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1155
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1156
  if (!is_static) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1157
    java_args.push_oop(receiver);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1158
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1159
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1160
  for (int i = 0; i < args_len; i++) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1161
    oop type_mirror = ptypes->obj_at(i);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1162
    oop arg = args->obj_at(i);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1163
    if (java_lang_Class::is_primitive(type_mirror)) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1164
      jvalue value;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1165
      BasicType ptype = basic_type_mirror_to_basic_type(type_mirror, CHECK_NULL);
34666
1c7168ea0034 8140485: Class load and creation cleanup
mgronlun
parents: 34238
diff changeset
  1166
      BasicType atype = Reflection::unbox_for_primitive(arg, &value, CHECK_NULL);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1167
      if (ptype != atype) {
34666
1c7168ea0034 8140485: Class load and creation cleanup
mgronlun
parents: 34238
diff changeset
  1168
        Reflection::widen(&value, atype, ptype, CHECK_NULL);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1169
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1170
      switch (ptype) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1171
        case T_BOOLEAN:     java_args.push_int(value.z);    break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1172
        case T_CHAR:        java_args.push_int(value.c);    break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1173
        case T_BYTE:        java_args.push_int(value.b);    break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1174
        case T_SHORT:       java_args.push_int(value.s);    break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1175
        case T_INT:         java_args.push_int(value.i);    break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1176
        case T_LONG:        java_args.push_long(value.j);   break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1177
        case T_FLOAT:       java_args.push_float(value.f);  break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1178
        case T_DOUBLE:      java_args.push_double(value.d); break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1179
        default:
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1180
          THROW_MSG_0(vmSymbols::java_lang_IllegalArgumentException(), "argument type mismatch");
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1181
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1182
    } else {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1183
      if (arg != NULL) {
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 13391
diff changeset
  1184
        Klass* k = java_lang_Class::as_Klass(type_mirror);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1185
        if (!arg->is_a(k)) {
34666
1c7168ea0034 8140485: Class load and creation cleanup
mgronlun
parents: 34238
diff changeset
  1186
          THROW_MSG_0(vmSymbols::java_lang_IllegalArgumentException(),
1c7168ea0034 8140485: Class load and creation cleanup
mgronlun
parents: 34238
diff changeset
  1187
                      "argument type mismatch");
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1188
        }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1189
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1190
      Handle arg_handle(THREAD, arg);         // Create handle for argument
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1191
      java_args.push_oop(arg_handle); // Push handle
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1192
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1193
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1194
34666
1c7168ea0034 8140485: Class load and creation cleanup
mgronlun
parents: 34238
diff changeset
  1195
  assert(java_args.size_of_parameters() == method->size_of_parameters(),
1c7168ea0034 8140485: Class load and creation cleanup
mgronlun
parents: 34238
diff changeset
  1196
    "just checking");
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1197
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1198
  // All oops (including receiver) is passed in as Handles. An potential oop is returned as an
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1199
  // oop (i.e., NOT as an handle)
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1200
  JavaValue result(rtype);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1201
  JavaCalls::call(&result, method, &java_args, THREAD);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1202
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1203
  if (HAS_PENDING_EXCEPTION) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1204
    // Method threw an exception; wrap it in an InvocationTargetException
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1205
    oop target_exception = PENDING_EXCEPTION;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1206
    CLEAR_PENDING_EXCEPTION;
22897
77e1024a4a14 4505697: nsk/jdi/ExceptionEvent/_itself_/exevent006 and exevent008 tests fail with InvocationTargetException
jbachorik
parents: 22551
diff changeset
  1207
    // JVMTI has already reported the pending exception
77e1024a4a14 4505697: nsk/jdi/ExceptionEvent/_itself_/exevent006 and exevent008 tests fail with InvocationTargetException
jbachorik
parents: 22551
diff changeset
  1208
    // JVMTI internal flag reset is needed in order to report InvocationTargetException
77e1024a4a14 4505697: nsk/jdi/ExceptionEvent/_itself_/exevent006 and exevent008 tests fail with InvocationTargetException
jbachorik
parents: 22551
diff changeset
  1209
    if (THREAD->is_Java_thread()) {
34666
1c7168ea0034 8140485: Class load and creation cleanup
mgronlun
parents: 34238
diff changeset
  1210
      JvmtiExport::clear_detected_exception((JavaThread*)THREAD);
22897
77e1024a4a14 4505697: nsk/jdi/ExceptionEvent/_itself_/exevent006 and exevent008 tests fail with InvocationTargetException
jbachorik
parents: 22551
diff changeset
  1211
    }
77e1024a4a14 4505697: nsk/jdi/ExceptionEvent/_itself_/exevent006 and exevent008 tests fail with InvocationTargetException
jbachorik
parents: 22551
diff changeset
  1212
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1213
    JavaCallArguments args(Handle(THREAD, target_exception));
8076
96d498ec7ae1 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 7397
diff changeset
  1214
    THROW_ARG_0(vmSymbols::java_lang_reflect_InvocationTargetException(),
96d498ec7ae1 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 7397
diff changeset
  1215
                vmSymbols::throwable_void_signature(),
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1216
                &args);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1217
  } else {
31615
e48d94b97b6c 8130036: Fix problems with imprecise C++ coding.
goetz
parents: 31019
diff changeset
  1218
    if (rtype == T_BOOLEAN || rtype == T_BYTE || rtype == T_CHAR || rtype == T_SHORT) {
34666
1c7168ea0034 8140485: Class load and creation cleanup
mgronlun
parents: 34238
diff changeset
  1219
      narrow((jvalue*)result.get_value_addr(), rtype, CHECK_NULL);
31615
e48d94b97b6c 8130036: Fix problems with imprecise C++ coding.
goetz
parents: 31019
diff changeset
  1220
    }
34666
1c7168ea0034 8140485: Class load and creation cleanup
mgronlun
parents: 34238
diff changeset
  1221
    return Reflection::box((jvalue*)result.get_value_addr(), rtype, THREAD);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1222
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1223
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1224
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1225
// This would be nicer if, say, java.lang.reflect.Method was a subclass
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1226
// of java.lang.reflect.Constructor
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1227
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1228
oop Reflection::invoke_method(oop method_mirror, Handle receiver, objArrayHandle args, TRAPS) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1229
  oop mirror             = java_lang_reflect_Method::clazz(method_mirror);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1230
  int slot               = java_lang_reflect_Method::slot(method_mirror);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1231
  bool override          = java_lang_reflect_Method::override(method_mirror) != 0;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1232
  objArrayHandle ptypes(THREAD, objArrayOop(java_lang_reflect_Method::parameter_types(method_mirror)));
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1233
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1234
  oop return_type_mirror = java_lang_reflect_Method::return_type(method_mirror);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1235
  BasicType rtype;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1236
  if (java_lang_Class::is_primitive(return_type_mirror)) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1237
    rtype = basic_type_mirror_to_basic_type(return_type_mirror, CHECK_NULL);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1238
  } else {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1239
    rtype = T_OBJECT;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1240
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1241
46329
53ccc37bda19 8155672: Remove instanceKlassHandles and KlassHandles
coleenp
parents: 46289
diff changeset
  1242
  InstanceKlass* klass = InstanceKlass::cast(java_lang_Class::as_Klass(mirror));
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 13391
diff changeset
  1243
  Method* m = klass->method_with_idnum(slot);
224
6a257cd604e7 6667089: 3/3 multiple redefinitions of a class break reflection
dcubed
parents: 1
diff changeset
  1244
  if (m == NULL) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1245
    THROW_MSG_0(vmSymbols::java_lang_InternalError(), "invoke");
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1246
  }
224
6a257cd604e7 6667089: 3/3 multiple redefinitions of a class break reflection
dcubed
parents: 1
diff changeset
  1247
  methodHandle method(THREAD, m);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1248
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1249
  return invoke(klass, method, receiver, override, ptypes, rtype, args, true, THREAD);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1250
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1251
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1252
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1253
oop Reflection::invoke_constructor(oop constructor_mirror, objArrayHandle args, TRAPS) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1254
  oop mirror             = java_lang_reflect_Constructor::clazz(constructor_mirror);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1255
  int slot               = java_lang_reflect_Constructor::slot(constructor_mirror);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1256
  bool override          = java_lang_reflect_Constructor::override(constructor_mirror) != 0;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1257
  objArrayHandle ptypes(THREAD, objArrayOop(java_lang_reflect_Constructor::parameter_types(constructor_mirror)));
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1258
46329
53ccc37bda19 8155672: Remove instanceKlassHandles and KlassHandles
coleenp
parents: 46289
diff changeset
  1259
  InstanceKlass* klass = InstanceKlass::cast(java_lang_Class::as_Klass(mirror));
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 13391
diff changeset
  1260
  Method* m = klass->method_with_idnum(slot);
224
6a257cd604e7 6667089: 3/3 multiple redefinitions of a class break reflection
dcubed
parents: 1
diff changeset
  1261
  if (m == NULL) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1262
    THROW_MSG_0(vmSymbols::java_lang_InternalError(), "invoke");
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1263
  }
224
6a257cd604e7 6667089: 3/3 multiple redefinitions of a class break reflection
dcubed
parents: 1
diff changeset
  1264
  methodHandle method(THREAD, m);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1265
  assert(method->name() == vmSymbols::object_initializer_name(), "invalid constructor");
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1266
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1267
  // Make sure klass gets initialize
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1268
  klass->initialize(CHECK_NULL);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1269
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1270
  // Create new instance (the receiver)
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1271
  klass->check_valid_for_instantiation(false, CHECK_NULL);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1272
  Handle receiver = klass->allocate_instance_handle(CHECK_NULL);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1273
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1274
  // Ignore result from call and return receiver
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1275
  invoke(klass, method, receiver, override, ptypes, T_VOID, args, false, CHECK_NULL);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1276
  return receiver();
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1277
}