Thu, 04 Dec 2008 17:29:56 -0800
6739363: Xcheck jni doesn't check native function arguments
Summary: Fix adds support for verifying arguments with -Xcheck:jni.
Reviewed-by: coleenp
duke@435 | 1 | /* |
duke@435 | 2 | * Copyright 1998-2006 Sun Microsystems, Inc. All Rights Reserved. |
duke@435 | 3 | * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER. |
duke@435 | 4 | * |
duke@435 | 5 | * This code is free software; you can redistribute it and/or modify it |
duke@435 | 6 | * under the terms of the GNU General Public License version 2 only, as |
duke@435 | 7 | * published by the Free Software Foundation. |
duke@435 | 8 | * |
duke@435 | 9 | * This code is distributed in the hope that it will be useful, but WITHOUT |
duke@435 | 10 | * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or |
duke@435 | 11 | * FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License |
duke@435 | 12 | * version 2 for more details (a copy is included in the LICENSE file that |
duke@435 | 13 | * accompanied this code). |
duke@435 | 14 | * |
duke@435 | 15 | * You should have received a copy of the GNU General Public License version |
duke@435 | 16 | * 2 along with this work; if not, write to the Free Software Foundation, |
duke@435 | 17 | * Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA. |
duke@435 | 18 | * |
duke@435 | 19 | * Please contact Sun Microsystems, Inc., 4150 Network Circle, Santa Clara, |
duke@435 | 20 | * CA 95054 USA or visit www.sun.com if you need additional information or |
duke@435 | 21 | * have any questions. |
duke@435 | 22 | * |
duke@435 | 23 | */ |
duke@435 | 24 | |
duke@435 | 25 | # include "incls/_precompiled.incl" |
duke@435 | 26 | # include "incls/_hpi_windows.cpp.incl" |
duke@435 | 27 | |
duke@435 | 28 | typedef jint (JNICALL *init_t)(GetInterfaceFunc *, void *); |
duke@435 | 29 | |
duke@435 | 30 | void hpi::initialize_get_interface(vm_calls_t *callbacks) |
duke@435 | 31 | { |
duke@435 | 32 | // Build name of HPI. |
duke@435 | 33 | char lib_name[JVM_MAXPATHLEN]; |
duke@435 | 34 | |
duke@435 | 35 | if (HPILibPath && HPILibPath[0]) { |
duke@435 | 36 | strncpy(lib_name, HPILibPath, JVM_MAXPATHLEN - 1); |
duke@435 | 37 | lib_name[JVM_MAXPATHLEN - 1] = '\0'; |
duke@435 | 38 | } else { |
duke@435 | 39 | os::jvm_path(lib_name, sizeof lib_name); |
duke@435 | 40 | |
duke@435 | 41 | #ifdef PRODUCT |
duke@435 | 42 | const char *hpi_lib = "\\hpi.dll"; |
duke@435 | 43 | #else |
duke@435 | 44 | char *ptr = strrchr(lib_name, '\\'); |
duke@435 | 45 | // On Win98 GetModuleFileName() returns the path in the upper case. |
duke@435 | 46 | assert(_strnicmp(ptr, "\\jvm",4) == 0, "invalid library name"); |
duke@435 | 47 | const char *hpi_lib = (_strnicmp(ptr, "\\jvm_g",6) == 0) ? "\\hpi_g.dll" : "\\hpi.dll"; |
duke@435 | 48 | #endif |
duke@435 | 49 | |
duke@435 | 50 | *(::strrchr(lib_name, '\\')) = '\0'; /* get rid of "\\jvm.dll" */ |
duke@435 | 51 | char *p = ::strrchr(lib_name, '\\'); |
duke@435 | 52 | if (p != NULL) *p = '\0'; /* get rid of "\\hotspot" */ |
duke@435 | 53 | strcat(lib_name, hpi_lib); |
duke@435 | 54 | } |
duke@435 | 55 | |
duke@435 | 56 | // Load it. |
duke@435 | 57 | if (TraceHPI) tty->print_cr("Loading HPI %s ", lib_name); |
duke@435 | 58 | HINSTANCE lib_handle = LoadLibrary(lib_name); |
duke@435 | 59 | if (lib_handle == NULL) { |
duke@435 | 60 | if (TraceHPI) tty->print_cr("LoadLibrary failed, code = %d", GetLastError()); |
duke@435 | 61 | return; |
duke@435 | 62 | } |
duke@435 | 63 | |
duke@435 | 64 | // Find hpi initializer. |
duke@435 | 65 | init_t initer = (init_t)GetProcAddress(lib_handle, "DLL_Initialize"); |
duke@435 | 66 | if (initer == NULL) { |
duke@435 | 67 | if (TraceHPI) tty->print_cr("GetProcAddress failed, errcode = %d", GetLastError()); |
duke@435 | 68 | return; |
duke@435 | 69 | } |
duke@435 | 70 | |
duke@435 | 71 | // Call initializer. |
duke@435 | 72 | jint init_result = (*initer)(&_get_interface, callbacks); |
duke@435 | 73 | if (init_result < 0) { |
duke@435 | 74 | if (TraceHPI) tty->print_cr("DLL_Initialize failed, returned %ld", init_result); |
duke@435 | 75 | return; |
duke@435 | 76 | } |
duke@435 | 77 | |
duke@435 | 78 | if (TraceHPI) tty->print_cr("success"); |
duke@435 | 79 | return; |
duke@435 | 80 | } |