commit 3b0224dc3c2e7ffb93dc56970395003d0e387545
author destradaa <destradaa@google.com> Wed Jul 16 14:28:06 2014 -0700
committer destradaa <destradaa@google.com> Thu Jul 17 11:09:46 2014 -0700
tree 124600c9f791f7fc8df131c1a9cd07281df90985
parent 69eef8a0d250ae1b50b510499014e109bd2c6112

Fix exceptions in ActivityRecognition platform stack.

b/16348349

Change-Id: I1f85283c86130b86389a1be7da49904658b0558c

core/jni/android_hardware_location_ActivityRecognitionHardware.cpp

diff --git a/core/jni/android_hardware_location_ActivityRecognitionHardware.cpp b/core/jni/android_hardware_location_ActivityRecognitionHardware.cpp
index 5b542ba..b8fa04c 100644
--- a/core/jni/android_hardware_location_ActivityRecognitionHardware.cpp
+++ b/core/jni/android_hardware_location_ActivityRecognitionHardware.cpp
@@ -94,6 +94,7 @@
     JNIEnv* env = NULL;
     int result = attach_thread(&env);
     if (result != JNI_OK) {
+        ALOGE("Unable to attach thread with JNI.");
         return;
     }
 
@@ -215,7 +216,7 @@
         return NULL;
     }
 
-    jclass string_class = env->FindClass("java/lang/String;");
+    jclass string_class = env->FindClass("java/lang/String");
     if (string_class == NULL) {
         ALOGE("Unable to find String class for supported activities.");
         return NULL;
@@ -229,14 +230,8 @@
 
     for (int i = 0; i < list_size; ++i) {
         const char* string_ptr = const_cast<const char*>(list[i]);
-        jsize string_length = strlen(string_ptr);
-        jstring string = env->NewString((const jchar*) string_ptr, string_length);
+        jstring string = env->NewStringUTF(string_ptr);
         env->SetObjectArrayElement(string_array, i, string);
-
-        // log debugging information in case we need to try to trace issues with the strings
-        if (string_length) {
-            ALOGD("Invalid activity (index=%d) name size: %d", i, string_length);
-        }
     }
 
     return string_array;