8035829: [parfait] JNI exception pending in jdk/src/windows/native/sun/tools/attach/WindowsVirtualMachine.c
Reviewed-by: sundar, kevinw
This commit is contained in:
parent
80448466d4
commit
712d8ff48e
@ -388,6 +388,7 @@ JNIEXPORT void JNICALL Java_sun_tools_attach_WindowsVirtualMachine_enqueue
|
|||||||
if (argsLen > 0) {
|
if (argsLen > 0) {
|
||||||
if (argsLen > MAX_ARGS) {
|
if (argsLen > MAX_ARGS) {
|
||||||
JNU_ThrowInternalError(env, "Too many arguments");
|
JNU_ThrowInternalError(env, "Too many arguments");
|
||||||
|
return;
|
||||||
}
|
}
|
||||||
for (i=0; i<argsLen; i++) {
|
for (i=0; i<argsLen; i++) {
|
||||||
jobject obj = (*env)->GetObjectArrayElement(env, args, i);
|
jobject obj = (*env)->GetObjectArrayElement(env, args, i);
|
||||||
@ -423,6 +424,8 @@ JNIEXPORT void JNICALL Java_sun_tools_attach_WindowsVirtualMachine_enqueue
|
|||||||
stubLen = (DWORD)(*env)->GetArrayLength(env, stub);
|
stubLen = (DWORD)(*env)->GetArrayLength(env, stub);
|
||||||
stubCode = (*env)->GetByteArrayElements(env, stub, &isCopy);
|
stubCode = (*env)->GetByteArrayElements(env, stub, &isCopy);
|
||||||
|
|
||||||
|
if ((*env)->ExceptionOccurred(env)) return;
|
||||||
|
|
||||||
pCode = (PDWORD) VirtualAllocEx( hProcess, 0, stubLen, MEM_COMMIT, PAGE_EXECUTE_READWRITE );
|
pCode = (PDWORD) VirtualAllocEx( hProcess, 0, stubLen, MEM_COMMIT, PAGE_EXECUTE_READWRITE );
|
||||||
if (pCode == NULL) {
|
if (pCode == NULL) {
|
||||||
JNU_ThrowIOExceptionWithLastError(env, "VirtualAllocEx failed");
|
JNU_ThrowIOExceptionWithLastError(env, "VirtualAllocEx failed");
|
||||||
@ -592,6 +595,8 @@ static void jstring_to_cstring(JNIEnv* env, jstring jstr, char* cstr, int len) {
|
|||||||
cstr[0] = '\0';
|
cstr[0] = '\0';
|
||||||
} else {
|
} else {
|
||||||
str = JNU_GetStringPlatformChars(env, jstr, &isCopy);
|
str = JNU_GetStringPlatformChars(env, jstr, &isCopy);
|
||||||
|
if ((*env)->ExceptionOccurred(env)) return;
|
||||||
|
|
||||||
strncpy(cstr, str, len);
|
strncpy(cstr, str, len);
|
||||||
cstr[len-1] = '\0';
|
cstr[len-1] = '\0';
|
||||||
if (isCopy) {
|
if (isCopy) {
|
||||||
|
Loading…
Reference in New Issue
Block a user