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

8077407: Even better MBean connections

Browse Source 
Reviewed-by: sla
This commit is contained in:
Jaroslav Bachorik 2015-04-10 15:26:17 +02:00
parent 97a263953b
commit e63cd605f3
1 changed files with 10 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

20
jdk/src/java.management/share/classes/javax/management/MBeanServerInvocationHandler.java
View File

@ -141,12 +141,12 @@ public class MBeanServerInvocationHandler implements InvocationHandler {
if (connection == null) {
throw new IllegalArgumentException("Null connection");
}
// if (Proxy.isProxyClass(connection.getClass())) {
// if (MBeanServerInvocationHandler.class.isAssignableFrom(
// Proxy.getInvocationHandler(connection).getClass())) {
// throw new IllegalArgumentException("Wrapping MBeanServerInvocationHandler");
// }
// }
if (Proxy.isProxyClass(connection.getClass())) {
if (MBeanServerInvocationHandler.class.isAssignableFrom(
Proxy.getInvocationHandler(connection).getClass())) {
throw new IllegalArgumentException("Wrapping MBeanServerInvocationHandler");
}
}
if (objectName == null) {
throw new IllegalArgumentException("Null object name");
}
@ -424,10 +424,10 @@ public class MBeanServerInvocationHandler implements InvocationHandler {
new Class<?>[] {Object.class})
&& isLocal(proxy, method))
return true;
// if (methodName.equals("finalize")
// && method.getParameterTypes().length == 0) {
// return true;
// }
if (methodName.equals("finalize")
&& method.getParameterTypes().length == 0) {
return true;
}
return false;
}