Mercurial Mercurial > jdk-sandbox / changeset
summary | shortlog | changelog | graph | tags | bookmarks | branches | files | changeset | raw | help
Find changesets by keywords (author, files, the commit message), revision number or hash, or revset expression.
8019833: Wrong JNI error code for preexisting JVM
authorcoleenp
Wed, 03 Jul 2013 13:45:39 -0400
changeset 18680 6d5dc445baec
parent 18492 ab4fbcf75767
child 18682 694f6918ba42
8019833: Wrong JNI error code for preexisting JVM Summary: Return the appropriate JNI error message (instead of the generic one) when the JVM is already started Reviewed-by: coleenp, hseigel Contributed-by: sylvestre@debian.org
hotspot/src/share/vm/prims/jni.cpp file | annotate | diff | comparison | revisions 
--- a/hotspot/src/share/vm/prims/jni.cpp	Tue Jul 02 16:54:24 2013 +0200
+++ b/hotspot/src/share/vm/prims/jni.cpp	Wed Jul 03 13:45:39 2013 -0400
@@ -5097,7 +5097,7 @@
   // function used to determine this will always return false. Atomic::xchg
   // does not have this problem.
   if (Atomic::xchg(1, &vm_created) == 1) {
-    return JNI_ERR;   // already created, or create attempt in progress
+    return JNI_EEXIST;   // already created, or create attempt in progress
   }
   if (Atomic::xchg(0, &safe_to_recreate_vm) == 0) {
     return JNI_ERR;  // someone tried and failed and retry not allowed.
jdk-sandbox