[Java] add missing JNI exception checks to fix warnings (#2405)

* add missing JNI exception checks to fix warnings * updated based on comments * removed rethrow logic * replace tabs with spaces
microsoft · Sep 26, 2019 · 7b2963d · 7b2963d
1 parent a0a117a
commit 7b2963d
Showing 1 changed file with 6 additions and 0 deletions.
diff --git a/swig/lightgbmlib.i b/swig/lightgbmlib.i
@@ -156,7 +156,13 @@
 
       // get the size, indices and values
       auto indices = (jintArray)jenv->CallObjectMethod(objSparseVec, sparseVectorIndices);
+      if (jenv->ExceptionCheck()) {
+        return -1;
+      }
       auto values = (jdoubleArray)jenv->CallObjectMethod(objSparseVec, sparseVectorValues);
+      if (jenv->ExceptionCheck()) {
+        return -1;
+      }
       int size = jenv->GetArrayLength(indices);
 
       // Note: when testing on larger data (e.g. 288k rows per partition and 36mio rows total)