summaryrefslogtreecommitdiffstats
path: root/core/jni
diff options
context:
space:
mode:
authordestradaa <destradaa@google.com>2014-07-16 14:28:06 -0700
committerdestradaa <destradaa@google.com>2014-07-17 11:09:46 -0700
commit3b0224dc3c2e7ffb93dc56970395003d0e387545 (patch)
tree124600c9f791f7fc8df131c1a9cd07281df90985 /core/jni
parent69eef8a0d250ae1b50b510499014e109bd2c6112 (diff)
downloadframeworks_base-3b0224dc3c2e7ffb93dc56970395003d0e387545.zip
frameworks_base-3b0224dc3c2e7ffb93dc56970395003d0e387545.tar.gz
frameworks_base-3b0224dc3c2e7ffb93dc56970395003d0e387545.tar.bz2
Fix exceptions in ActivityRecognition platform stack.
b/16348349 Change-Id: I1f85283c86130b86389a1be7da49904658b0558c
Diffstat (limited to 'core/jni')
-rw-r--r--core/jni/android_hardware_location_ActivityRecognitionHardware.cpp11
1 files changed, 3 insertions, 8 deletions
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 @@ static void activity_callback(
JNIEnv* env = NULL;
int result = attach_thread(&env);
if (result != JNI_OK) {
+ ALOGE("Unable to attach thread with JNI.");
return;
}
@@ -215,7 +216,7 @@ static jobjectArray get_supported_activities(JNIEnv* env, jobject obj) {
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 @@ static jobjectArray get_supported_activities(JNIEnv* env, jobject obj) {
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;