7172551: Remove Native calls from DriverManager for jigsaw
Reviewed-by: alanb, chegar, darcy, mchung
--- a/jdk/make/java/java/FILES_c.gmk Thu Jun 07 01:01:09 2012 -0700
+++ b/jdk/make/java/java/FILES_c.gmk Thu Jun 07 20:11:26 2012 -0400
@@ -69,7 +69,6 @@
jdk_util_md.c \
check_version.c \
java_props_md.c \
- DriverManager.c \
ConstantPool.c \
MessageUtils.c \
GC.c \
--- a/jdk/make/java/java/mapfile-vers Thu Jun 07 01:01:09 2012 -0700
+++ b/jdk/make/java/java/mapfile-vers Thu Jun 07 20:11:26 2012 -0400
@@ -232,7 +232,6 @@
Java_java_security_AccessController_doPrivileged__Ljava_security_PrivilegedExceptionAction_2Ljava_security_AccessControlContext_2;
Java_java_security_AccessController_getStackAccessControlContext;
Java_java_security_AccessController_getInheritedAccessControlContext;
- Java_java_sql_DriverManager_getCallerClassLoader;
Java_java_util_ResourceBundle_getClassContext;
Java_java_util_TimeZone_getSystemTimeZoneID;
Java_java_util_TimeZone_getSystemGMTOffsetID;
--- a/jdk/makefiles/java/java/FILES_c.gmk Thu Jun 07 01:01:09 2012 -0700
+++ b/jdk/makefiles/java/java/FILES_c.gmk Thu Jun 07 20:11:26 2012 -0400
@@ -69,7 +69,6 @@
jdk_util_md.c \
check_version.c \
java_props_md.c \
- DriverManager.c \
ConstantPool.c \
MessageUtils.c \
GC.c \
--- a/jdk/makefiles/java/java/mapfile-vers Thu Jun 07 01:01:09 2012 -0700
+++ b/jdk/makefiles/java/java/mapfile-vers Thu Jun 07 20:11:26 2012 -0400
@@ -232,7 +232,6 @@
Java_java_security_AccessController_doPrivileged__Ljava_security_PrivilegedExceptionAction_2Ljava_security_AccessControlContext_2;
Java_java_security_AccessController_getStackAccessControlContext;
Java_java_security_AccessController_getInheritedAccessControlContext;
- Java_java_sql_DriverManager_getCallerClassLoader;
Java_java_util_ResourceBundle_getClassContext;
Java_java_util_TimeZone_getSystemTimeZoneID;
Java_java_util_TimeZone_getSystemGMTOffsetID;
--- a/jdk/src/share/classes/java/sql/DriverManager.java Thu Jun 07 01:01:09 2012 -0700
+++ b/jdk/src/share/classes/java/sql/DriverManager.java Thu Jun 07 20:11:26 2012 -0400
@@ -1,5 +1,5 @@
/*
- * Copyright (c) 1996, 2011, Oracle and/or its affiliates. All rights reserved.
+ * Copyright (c) 1996, 2012, Oracle and/or its affiliates. All rights reserved.
* DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
*
* This code is free software; you can redistribute it and/or modify it
@@ -30,6 +30,7 @@
import java.security.AccessController;
import java.security.PrivilegedAction;
import java.util.concurrent.CopyOnWriteArrayList;
+import sun.reflect.Reflection;
/**
@@ -462,6 +463,15 @@
//------------------------------------------------------------------------
+ // Internal method used to get the caller's class loader.
+ // Replaces the call to the native method
+ private static ClassLoader getCallerClassLoader() {
+ Class<?> cc = Reflection.getCallerClass(3);
+ ClassLoader cl = (cc != null) ? cc.getClassLoader() : null;
+ return cl;
+ }
+
+
// Indicates whether the class object that would be created if the code calling
// DriverManager is accessible.
private static boolean isDriverAllowed(Driver driver, ClassLoader classLoader) {
@@ -604,8 +614,6 @@
throw new SQLException("No suitable driver found for "+ url, "08001");
}
- /* Returns the caller's class loader, or null if none */
- private static native ClassLoader getCallerClassLoader();
}
--- a/jdk/src/share/native/java/sql/DriverManager.c Thu Jun 07 01:01:09 2012 -0700
+++ /dev/null Thu Jan 01 00:00:00 1970 +0000
@@ -1,37 +0,0 @@
-/*
- * Copyright (c) 1998, Oracle and/or its affiliates. All rights reserved.
- * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
- *
- * This code is free software; you can redistribute it and/or modify it
- * under the terms of the GNU General Public License version 2 only, as
- * published by the Free Software Foundation. Oracle designates this
- * particular file as subject to the "Classpath" exception as provided
- * by Oracle in the LICENSE file that accompanied this code.
- *
- * This code is distributed in the hope that it will be useful, but WITHOUT
- * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
- * FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License
- * version 2 for more details (a copy is included in the LICENSE file that
- * accompanied this code).
- *
- * You should have received a copy of the GNU General Public License version
- * 2 along with this work; if not, write to the Free Software Foundation,
- * Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA.
- *
- * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA
- * or visit www.oracle.com if you need additional information or have any
- * questions.
- */
-
-#include <stdlib.h>
-
-#include "jni.h"
-#include "jni_util.h"
-#include "jvm.h"
-
-JNIEXPORT jobject JNICALL
-Java_java_sql_DriverManager_getCallerClassLoader(JNIEnv *env, jobject this)
-{
- jclass caller = JVM_GetCallerClass(env, 2);
- return caller != 0 ? JVM_GetClassLoader(env, caller) : 0;
-}