diff options
author | Brian Carlstrom <bdc@google.com> | 2011-04-05 15:20:52 -0700 |
---|---|---|
committer | Brian Carlstrom <bdc@google.com> | 2011-04-05 15:20:52 -0700 |
commit | b32b1ddf722a94962321f78c8cd87c6924e1e9a9 (patch) | |
tree | 66d9c13ebd055c3177def0e14f9f1aa2b78eb8e9 | |
parent | 42bc0e946f8b986fb3aaada9980b496172e2b511 (diff) | |
download | frameworks_base-b32b1ddf722a94962321f78c8cd87c6924e1e9a9.zip frameworks_base-b32b1ddf722a94962321f78c8cd87c6924e1e9a9.tar.gz frameworks_base-b32b1ddf722a94962321f78c8cd87c6924e1e9a9.tar.bz2 |
Fix usages of PopLocalFrame to include returned local reference
Change-Id: Ibd7a37b4035862718cb625110cfbba741537c17a
-rw-r--r-- | core/java/android/server/BluetoothAdapterProperties.java | 3 | ||||
-rw-r--r-- | core/jni/android_server_BluetoothService.cpp | 7 |
2 files changed, 3 insertions, 7 deletions
diff --git a/core/java/android/server/BluetoothAdapterProperties.java b/core/java/android/server/BluetoothAdapterProperties.java index ae8104b..9723f60 100644 --- a/core/java/android/server/BluetoothAdapterProperties.java +++ b/core/java/android/server/BluetoothAdapterProperties.java @@ -76,14 +76,13 @@ class BluetoothAdapterProperties { for (int i = 0; i < properties.length; i++) { String name = properties[i]; String newValue = null; - int len; if (name == null) { Log.e(TAG, "Error:Adapter Property at index " + i + " is null"); continue; } if (name.equals("Devices") || name.equals("UUIDs")) { StringBuilder str = new StringBuilder(); - len = Integer.valueOf(properties[++i]); + int len = Integer.valueOf(properties[++i]); for (int j = 0; j < len; j++) { str.append(properties[++i]); str.append(","); diff --git a/core/jni/android_server_BluetoothService.cpp b/core/jni/android_server_BluetoothService.cpp index 158e475..5c6958a 100644 --- a/core/jni/android_server_BluetoothService.cpp +++ b/core/jni/android_server_BluetoothService.cpp @@ -695,9 +695,7 @@ static jobjectArray getDevicePropertiesNative(JNIEnv *env, jobject object, str_array = parse_remote_device_properties(env, &iter); dbus_message_unref(reply); - env->PopLocalFrame(NULL); - - return str_array; + return (jobjectArray) env->PopLocalFrame(str_array); } #endif return NULL; @@ -731,8 +729,7 @@ static jobjectArray getAdapterPropertiesNative(JNIEnv *env, jobject object) { str_array = parse_adapter_properties(env, &iter); dbus_message_unref(reply); - env->PopLocalFrame(NULL); - return str_array; + return (jobjectArray) env->PopLocalFrame(str_array); } #endif return NULL; |