annotate agent/src/share/classes/sun/jvm/hotspot/jdi/InterfaceTypeImpl.java @ 14188:29985fccf378

4990369: visibleMethods() and methodsByName() return wrong visible methods Reviewed-by: sspitsyn, coleenp
author sla
date Tue, 07 Jan 2014 10:50:28 +0100
parents c18cbe5936b8
children 4ca6dc0799b6
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
rev   line source
0
a61af66fc99e Initial load
duke
parents:
diff changeset
1 /*
1552
c18cbe5936b8 6941466: Oracle rebranding changes for Hotspot repositories
trims
parents: 0
diff changeset
2 * Copyright (c) 2002, 2004, Oracle and/or its affiliates. All rights reserved.
0
a61af66fc99e Initial load
duke
parents:
diff changeset
3 * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
a61af66fc99e Initial load
duke
parents:
diff changeset
4 *
a61af66fc99e Initial load
duke
parents:
diff changeset
5 * This code is free software; you can redistribute it and/or modify it
a61af66fc99e Initial load
duke
parents:
diff changeset
6 * under the terms of the GNU General Public License version 2 only, as
a61af66fc99e Initial load
duke
parents:
diff changeset
7 * published by the Free Software Foundation.
a61af66fc99e Initial load
duke
parents:
diff changeset
8 *
a61af66fc99e Initial load
duke
parents:
diff changeset
9 * This code is distributed in the hope that it will be useful, but WITHOUT
a61af66fc99e Initial load
duke
parents:
diff changeset
10 * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
a61af66fc99e Initial load
duke
parents:
diff changeset
11 * FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License
a61af66fc99e Initial load
duke
parents:
diff changeset
12 * version 2 for more details (a copy is included in the LICENSE file that
a61af66fc99e Initial load
duke
parents:
diff changeset
13 * accompanied this code).
a61af66fc99e Initial load
duke
parents:
diff changeset
14 *
a61af66fc99e Initial load
duke
parents:
diff changeset
15 * You should have received a copy of the GNU General Public License version
a61af66fc99e Initial load
duke
parents:
diff changeset
16 * 2 along with this work; if not, write to the Free Software Foundation,
a61af66fc99e Initial load
duke
parents:
diff changeset
17 * Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA.
a61af66fc99e Initial load
duke
parents:
diff changeset
18 *
1552
c18cbe5936b8 6941466: Oracle rebranding changes for Hotspot repositories
trims
parents: 0
diff changeset
19 * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA
c18cbe5936b8 6941466: Oracle rebranding changes for Hotspot repositories
trims
parents: 0
diff changeset
20 * or visit www.oracle.com if you need additional information or have any
c18cbe5936b8 6941466: Oracle rebranding changes for Hotspot repositories
trims
parents: 0
diff changeset
21 * questions.
0
a61af66fc99e Initial load
duke
parents:
diff changeset
22 *
a61af66fc99e Initial load
duke
parents:
diff changeset
23 */
a61af66fc99e Initial load
duke
parents:
diff changeset
24
a61af66fc99e Initial load
duke
parents:
diff changeset
25 package sun.jvm.hotspot.jdi;
a61af66fc99e Initial load
duke
parents:
diff changeset
26
14188
29985fccf378 4990369: visibleMethods() and methodsByName() return wrong visible methods
sla
parents: 1552
diff changeset
27 import java.lang.ref.SoftReference;
29985fccf378 4990369: visibleMethods() and methodsByName() return wrong visible methods
sla
parents: 1552
diff changeset
28 import java.util.ArrayList;
29985fccf378 4990369: visibleMethods() and methodsByName() return wrong visible methods
sla
parents: 1552
diff changeset
29 import java.util.Collections;
29985fccf378 4990369: visibleMethods() and methodsByName() return wrong visible methods
sla
parents: 1552
diff changeset
30 import java.util.Iterator;
29985fccf378 4990369: visibleMethods() and methodsByName() return wrong visible methods
sla
parents: 1552
diff changeset
31 import java.util.List;
29985fccf378 4990369: visibleMethods() and methodsByName() return wrong visible methods
sla
parents: 1552
diff changeset
32 import java.util.Map;
29985fccf378 4990369: visibleMethods() and methodsByName() return wrong visible methods
sla
parents: 1552
diff changeset
33 import java.util.Set;
29985fccf378 4990369: visibleMethods() and methodsByName() return wrong visible methods
sla
parents: 1552
diff changeset
34
0
a61af66fc99e Initial load
duke
parents:
diff changeset
35 import sun.jvm.hotspot.oops.InstanceKlass;
a61af66fc99e Initial load
duke
parents:
diff changeset
36
14188
29985fccf378 4990369: visibleMethods() and methodsByName() return wrong visible methods
sla
parents: 1552
diff changeset
37 import com.sun.jdi.ClassNotPreparedException;
29985fccf378 4990369: visibleMethods() and methodsByName() return wrong visible methods
sla
parents: 1552
diff changeset
38 import com.sun.jdi.ClassType;
29985fccf378 4990369: visibleMethods() and methodsByName() return wrong visible methods
sla
parents: 1552
diff changeset
39 import com.sun.jdi.InterfaceType;
29985fccf378 4990369: visibleMethods() and methodsByName() return wrong visible methods
sla
parents: 1552
diff changeset
40 import com.sun.jdi.Method;
29985fccf378 4990369: visibleMethods() and methodsByName() return wrong visible methods
sla
parents: 1552
diff changeset
41 import com.sun.jdi.ReferenceType;
29985fccf378 4990369: visibleMethods() and methodsByName() return wrong visible methods
sla
parents: 1552
diff changeset
42 import com.sun.jdi.VirtualMachine;
0
a61af66fc99e Initial load
duke
parents:
diff changeset
43
a61af66fc99e Initial load
duke
parents:
diff changeset
44 public class InterfaceTypeImpl extends ReferenceTypeImpl
a61af66fc99e Initial load
duke
parents:
diff changeset
45 implements InterfaceType {
a61af66fc99e Initial load
duke
parents:
diff changeset
46 private SoftReference superInterfacesCache = null;
a61af66fc99e Initial load
duke
parents:
diff changeset
47 private SoftReference subInterfacesCache = null;
a61af66fc99e Initial load
duke
parents:
diff changeset
48 private SoftReference implementorsCache = null;
a61af66fc99e Initial load
duke
parents:
diff changeset
49
a61af66fc99e Initial load
duke
parents:
diff changeset
50 protected InterfaceTypeImpl(VirtualMachine aVm, InstanceKlass aRef) {
a61af66fc99e Initial load
duke
parents:
diff changeset
51 super(aVm, aRef);
a61af66fc99e Initial load
duke
parents:
diff changeset
52 }
a61af66fc99e Initial load
duke
parents:
diff changeset
53
a61af66fc99e Initial load
duke
parents:
diff changeset
54 public List superinterfaces() throws ClassNotPreparedException {
a61af66fc99e Initial load
duke
parents:
diff changeset
55 List superinterfaces = (superInterfacesCache != null)? (List) superInterfacesCache.get() : null;
a61af66fc99e Initial load
duke
parents:
diff changeset
56 if (superinterfaces == null) {
a61af66fc99e Initial load
duke
parents:
diff changeset
57 checkPrepared();
a61af66fc99e Initial load
duke
parents:
diff changeset
58 superinterfaces = Collections.unmodifiableList(getInterfaces());
a61af66fc99e Initial load
duke
parents:
diff changeset
59 superInterfacesCache = new SoftReference(superinterfaces);
a61af66fc99e Initial load
duke
parents:
diff changeset
60 }
a61af66fc99e Initial load
duke
parents:
diff changeset
61 return superinterfaces;
a61af66fc99e Initial load
duke
parents:
diff changeset
62 }
a61af66fc99e Initial load
duke
parents:
diff changeset
63
a61af66fc99e Initial load
duke
parents:
diff changeset
64 public List subinterfaces() {
a61af66fc99e Initial load
duke
parents:
diff changeset
65 List subinterfaces = (subInterfacesCache != null)? (List) subInterfacesCache.get() : null;
a61af66fc99e Initial load
duke
parents:
diff changeset
66 if (subinterfaces == null) {
a61af66fc99e Initial load
duke
parents:
diff changeset
67 List all = vm.allClasses();
a61af66fc99e Initial load
duke
parents:
diff changeset
68 subinterfaces = new ArrayList();
a61af66fc99e Initial load
duke
parents:
diff changeset
69 Iterator iter = all.iterator();
a61af66fc99e Initial load
duke
parents:
diff changeset
70 while (iter.hasNext()) {
a61af66fc99e Initial load
duke
parents:
diff changeset
71 ReferenceType refType = (ReferenceType)iter.next();
a61af66fc99e Initial load
duke
parents:
diff changeset
72 if (refType instanceof InterfaceType) {
a61af66fc99e Initial load
duke
parents:
diff changeset
73 InterfaceType interfaze = (InterfaceType)refType;
a61af66fc99e Initial load
duke
parents:
diff changeset
74 if (interfaze.isPrepared() && interfaze.superinterfaces().contains(this)) {
a61af66fc99e Initial load
duke
parents:
diff changeset
75 subinterfaces.add(interfaze);
a61af66fc99e Initial load
duke
parents:
diff changeset
76 }
a61af66fc99e Initial load
duke
parents:
diff changeset
77 }
a61af66fc99e Initial load
duke
parents:
diff changeset
78 }
a61af66fc99e Initial load
duke
parents:
diff changeset
79 subinterfaces = Collections.unmodifiableList(subinterfaces);
a61af66fc99e Initial load
duke
parents:
diff changeset
80 subInterfacesCache = new SoftReference(subinterfaces);
a61af66fc99e Initial load
duke
parents:
diff changeset
81 }
a61af66fc99e Initial load
duke
parents:
diff changeset
82 return subinterfaces;
a61af66fc99e Initial load
duke
parents:
diff changeset
83 }
a61af66fc99e Initial load
duke
parents:
diff changeset
84
a61af66fc99e Initial load
duke
parents:
diff changeset
85 public List implementors() {
a61af66fc99e Initial load
duke
parents:
diff changeset
86 List implementors = (implementorsCache != null)? (List) implementorsCache.get() : null;
a61af66fc99e Initial load
duke
parents:
diff changeset
87 if (implementors == null) {
a61af66fc99e Initial load
duke
parents:
diff changeset
88 List all = vm.allClasses();
a61af66fc99e Initial load
duke
parents:
diff changeset
89 implementors = new ArrayList();
a61af66fc99e Initial load
duke
parents:
diff changeset
90 Iterator iter = all.iterator();
a61af66fc99e Initial load
duke
parents:
diff changeset
91 while (iter.hasNext()) {
a61af66fc99e Initial load
duke
parents:
diff changeset
92 ReferenceType refType = (ReferenceType)iter.next();
a61af66fc99e Initial load
duke
parents:
diff changeset
93 if (refType instanceof ClassType) {
a61af66fc99e Initial load
duke
parents:
diff changeset
94 ClassType clazz = (ClassType)refType;
a61af66fc99e Initial load
duke
parents:
diff changeset
95 if (clazz.isPrepared() && clazz.interfaces().contains(this)) {
a61af66fc99e Initial load
duke
parents:
diff changeset
96 implementors.add(clazz);
a61af66fc99e Initial load
duke
parents:
diff changeset
97 }
a61af66fc99e Initial load
duke
parents:
diff changeset
98 }
a61af66fc99e Initial load
duke
parents:
diff changeset
99 }
a61af66fc99e Initial load
duke
parents:
diff changeset
100 implementors = Collections.unmodifiableList(implementors);
a61af66fc99e Initial load
duke
parents:
diff changeset
101 implementorsCache = new SoftReference(implementors);
a61af66fc99e Initial load
duke
parents:
diff changeset
102 }
a61af66fc99e Initial load
duke
parents:
diff changeset
103 return implementors;
a61af66fc99e Initial load
duke
parents:
diff changeset
104 }
a61af66fc99e Initial load
duke
parents:
diff changeset
105
14188
29985fccf378 4990369: visibleMethods() and methodsByName() return wrong visible methods
sla
parents: 1552
diff changeset
106 @Override
29985fccf378 4990369: visibleMethods() and methodsByName() return wrong visible methods
sla
parents: 1552
diff changeset
107 void addVisibleMethods(Map<String, Method> methodMap, Set<InterfaceType> seenInterfaces) {
0
a61af66fc99e Initial load
duke
parents:
diff changeset
108 /*
a61af66fc99e Initial load
duke
parents:
diff changeset
109 * Add methods from
a61af66fc99e Initial load
duke
parents:
diff changeset
110 * parent types first, so that the methods in this class will
a61af66fc99e Initial load
duke
parents:
diff changeset
111 * overwrite them in the hash table
a61af66fc99e Initial load
duke
parents:
diff changeset
112 */
14188
29985fccf378 4990369: visibleMethods() and methodsByName() return wrong visible methods
sla
parents: 1552
diff changeset
113 Iterator<InterfaceType> iter = superinterfaces().iterator();
0
a61af66fc99e Initial load
duke
parents:
diff changeset
114 while (iter.hasNext()) {
a61af66fc99e Initial load
duke
parents:
diff changeset
115 InterfaceTypeImpl interfaze = (InterfaceTypeImpl)iter.next();
14188
29985fccf378 4990369: visibleMethods() and methodsByName() return wrong visible methods
sla
parents: 1552
diff changeset
116 if (!seenInterfaces.contains(interfaze)) {
29985fccf378 4990369: visibleMethods() and methodsByName() return wrong visible methods
sla
parents: 1552
diff changeset
117 interfaze.addVisibleMethods(methodMap, seenInterfaces);
29985fccf378 4990369: visibleMethods() and methodsByName() return wrong visible methods
sla
parents: 1552
diff changeset
118 seenInterfaces.add(interfaze);
29985fccf378 4990369: visibleMethods() and methodsByName() return wrong visible methods
sla
parents: 1552
diff changeset
119 }
0
a61af66fc99e Initial load
duke
parents:
diff changeset
120 }
a61af66fc99e Initial load
duke
parents:
diff changeset
121
a61af66fc99e Initial load
duke
parents:
diff changeset
122 addToMethodMap(methodMap, methods());
a61af66fc99e Initial load
duke
parents:
diff changeset
123 }
a61af66fc99e Initial load
duke
parents:
diff changeset
124
a61af66fc99e Initial load
duke
parents:
diff changeset
125 List getAllMethods() {
a61af66fc99e Initial load
duke
parents:
diff changeset
126 ArrayList list = new ArrayList(methods());
a61af66fc99e Initial load
duke
parents:
diff changeset
127 /*
a61af66fc99e Initial load
duke
parents:
diff changeset
128 * It's more efficient if don't do this
a61af66fc99e Initial load
duke
parents:
diff changeset
129 * recursively.
a61af66fc99e Initial load
duke
parents:
diff changeset
130 */
a61af66fc99e Initial load
duke
parents:
diff changeset
131 List interfaces = allSuperinterfaces();
a61af66fc99e Initial load
duke
parents:
diff changeset
132 Iterator iter = interfaces.iterator();
a61af66fc99e Initial load
duke
parents:
diff changeset
133 while (iter.hasNext()) {
a61af66fc99e Initial load
duke
parents:
diff changeset
134 InterfaceType interfaze = (InterfaceType)iter.next();
a61af66fc99e Initial load
duke
parents:
diff changeset
135 list.addAll(interfaze.methods());
a61af66fc99e Initial load
duke
parents:
diff changeset
136 }
a61af66fc99e Initial load
duke
parents:
diff changeset
137
a61af66fc99e Initial load
duke
parents:
diff changeset
138 return list;
a61af66fc99e Initial load
duke
parents:
diff changeset
139 }
a61af66fc99e Initial load
duke
parents:
diff changeset
140
a61af66fc99e Initial load
duke
parents:
diff changeset
141 List allSuperinterfaces() {
a61af66fc99e Initial load
duke
parents:
diff changeset
142 ArrayList list = new ArrayList();
a61af66fc99e Initial load
duke
parents:
diff changeset
143 addSuperinterfaces(list);
a61af66fc99e Initial load
duke
parents:
diff changeset
144 return list;
a61af66fc99e Initial load
duke
parents:
diff changeset
145 }
a61af66fc99e Initial load
duke
parents:
diff changeset
146
a61af66fc99e Initial load
duke
parents:
diff changeset
147 void addSuperinterfaces(List list) {
a61af66fc99e Initial load
duke
parents:
diff changeset
148 /*
a61af66fc99e Initial load
duke
parents:
diff changeset
149 * This code is a little strange because it
a61af66fc99e Initial load
duke
parents:
diff changeset
150 * builds the list with a more suitable order than the
a61af66fc99e Initial load
duke
parents:
diff changeset
151 * depth-first approach a normal recursive solution would
a61af66fc99e Initial load
duke
parents:
diff changeset
152 * take. Instead, all direct superinterfaces precede all
a61af66fc99e Initial load
duke
parents:
diff changeset
153 * indirect ones.
a61af66fc99e Initial load
duke
parents:
diff changeset
154 */
a61af66fc99e Initial load
duke
parents:
diff changeset
155
a61af66fc99e Initial load
duke
parents:
diff changeset
156 /*
a61af66fc99e Initial load
duke
parents:
diff changeset
157 * Get a list of direct superinterfaces that's not already in the
a61af66fc99e Initial load
duke
parents:
diff changeset
158 * list being built.
a61af66fc99e Initial load
duke
parents:
diff changeset
159 */
a61af66fc99e Initial load
duke
parents:
diff changeset
160 List immediate = new ArrayList(superinterfaces());
a61af66fc99e Initial load
duke
parents:
diff changeset
161 Iterator iter = immediate.iterator();
a61af66fc99e Initial load
duke
parents:
diff changeset
162 while (iter.hasNext()) {
a61af66fc99e Initial load
duke
parents:
diff changeset
163 InterfaceType interfaze = (InterfaceType)iter.next();
a61af66fc99e Initial load
duke
parents:
diff changeset
164 if (list.contains(interfaze)) {
a61af66fc99e Initial load
duke
parents:
diff changeset
165 iter.remove();
a61af66fc99e Initial load
duke
parents:
diff changeset
166 }
a61af66fc99e Initial load
duke
parents:
diff changeset
167 }
a61af66fc99e Initial load
duke
parents:
diff changeset
168
a61af66fc99e Initial load
duke
parents:
diff changeset
169 /*
a61af66fc99e Initial load
duke
parents:
diff changeset
170 * Add all new direct superinterfaces
a61af66fc99e Initial load
duke
parents:
diff changeset
171 */
a61af66fc99e Initial load
duke
parents:
diff changeset
172 list.addAll(immediate);
a61af66fc99e Initial load
duke
parents:
diff changeset
173
a61af66fc99e Initial load
duke
parents:
diff changeset
174 /*
a61af66fc99e Initial load
duke
parents:
diff changeset
175 * Recurse for all new direct superinterfaces.
a61af66fc99e Initial load
duke
parents:
diff changeset
176 */
a61af66fc99e Initial load
duke
parents:
diff changeset
177 iter = immediate.iterator();
a61af66fc99e Initial load
duke
parents:
diff changeset
178 while (iter.hasNext()) {
a61af66fc99e Initial load
duke
parents:
diff changeset
179 InterfaceTypeImpl interfaze = (InterfaceTypeImpl)iter.next();
a61af66fc99e Initial load
duke
parents:
diff changeset
180 interfaze.addSuperinterfaces(list);
a61af66fc99e Initial load
duke
parents:
diff changeset
181 }
a61af66fc99e Initial load
duke
parents:
diff changeset
182 }
a61af66fc99e Initial load
duke
parents:
diff changeset
183
a61af66fc99e Initial load
duke
parents:
diff changeset
184 boolean isAssignableTo(ReferenceType type) {
a61af66fc99e Initial load
duke
parents:
diff changeset
185
a61af66fc99e Initial load
duke
parents:
diff changeset
186 // Exact match?
a61af66fc99e Initial load
duke
parents:
diff changeset
187 if (this.equals(type)) {
a61af66fc99e Initial load
duke
parents:
diff changeset
188 return true;
a61af66fc99e Initial load
duke
parents:
diff changeset
189 } else {
a61af66fc99e Initial load
duke
parents:
diff changeset
190 // Try superinterfaces.
a61af66fc99e Initial load
duke
parents:
diff changeset
191 List supers = superinterfaces();
a61af66fc99e Initial load
duke
parents:
diff changeset
192 Iterator iter = supers.iterator();
a61af66fc99e Initial load
duke
parents:
diff changeset
193 while (iter.hasNext()) {
a61af66fc99e Initial load
duke
parents:
diff changeset
194 InterfaceTypeImpl interfaze = (InterfaceTypeImpl)iter.next();
a61af66fc99e Initial load
duke
parents:
diff changeset
195 if (interfaze.isAssignableTo(type)) {
a61af66fc99e Initial load
duke
parents:
diff changeset
196 return true;
a61af66fc99e Initial load
duke
parents:
diff changeset
197 }
a61af66fc99e Initial load
duke
parents:
diff changeset
198 }
a61af66fc99e Initial load
duke
parents:
diff changeset
199
a61af66fc99e Initial load
duke
parents:
diff changeset
200 return false;
a61af66fc99e Initial load
duke
parents:
diff changeset
201 }
a61af66fc99e Initial load
duke
parents:
diff changeset
202 }
a61af66fc99e Initial load
duke
parents:
diff changeset
203
a61af66fc99e Initial load
duke
parents:
diff changeset
204 List inheritedTypes() {
a61af66fc99e Initial load
duke
parents:
diff changeset
205 return superinterfaces();
a61af66fc99e Initial load
duke
parents:
diff changeset
206 }
a61af66fc99e Initial load
duke
parents:
diff changeset
207
a61af66fc99e Initial load
duke
parents:
diff changeset
208 public boolean isInitialized() {
a61af66fc99e Initial load
duke
parents:
diff changeset
209 return isPrepared();
a61af66fc99e Initial load
duke
parents:
diff changeset
210 }
a61af66fc99e Initial load
duke
parents:
diff changeset
211
a61af66fc99e Initial load
duke
parents:
diff changeset
212 public String toString() {
a61af66fc99e Initial load
duke
parents:
diff changeset
213 return "interface " + name() + " (" + loaderString() + ")";
a61af66fc99e Initial load
duke
parents:
diff changeset
214 }
a61af66fc99e Initial load
duke
parents:
diff changeset
215 }