diff options
author | Nancy Chen <nancychen@google.com> | 2015-03-23 15:03:03 -0700 |
---|---|---|
committer | Nancy Chen <nancychen@google.com> | 2015-03-23 15:11:00 -0700 |
commit | 0f4ec2a7c965a8fa23fa32373fbdbbb3e4be1ea4 (patch) | |
tree | 9850054ad79cd76bcf03d3e28cd74065ded51d1f | |
parent | 72c9053eda382ef3e4450b657801898aced8f904 (diff) |
Fix NPE in device ID dialog.
Do not add null strings to list of device ids when responding to MMI
code for device ids (IMEI/MEID). Also fix string resource that is not
available in AOSP.
Bug: 19897433
Change-Id: Ie06bb793eafa66f451bf1843d2a5ad4d343999d2
-rw-r--r-- | src/com/android/dialer/SpecialCharSequenceMgr.java | 8 |
1 files changed, 6 insertions, 2 deletions
diff --git a/src/com/android/dialer/SpecialCharSequenceMgr.java b/src/com/android/dialer/SpecialCharSequenceMgr.java index f466c417c..46c6de6c9 100644 --- a/src/com/android/dialer/SpecialCharSequenceMgr.java +++ b/src/com/android/dialer/SpecialCharSequenceMgr.java @@ -33,6 +33,7 @@ import android.telecom.PhoneAccountHandle; import android.telecom.TelecomManager; import android.telephony.PhoneNumberUtils; import android.telephony.TelephonyManager; +import android.text.TextUtils; import android.util.Log; import android.view.WindowManager; import android.widget.EditText; @@ -310,13 +311,16 @@ public class SpecialCharSequenceMgr { List<String> deviceIds = new ArrayList<String>(); for (int slot = 0; slot < telephonyManager.getPhoneCount(); slot++) { - deviceIds.add(telephonyManager.getDeviceId(slot)); + String deviceId = telephonyManager.getDeviceId(slot); + if (!TextUtils.isEmpty(deviceId)) { + deviceIds.add(deviceId); + } } AlertDialog alert = new AlertDialog.Builder(context) .setTitle(labelResId) .setItems(deviceIds.toArray(new String[deviceIds.size()]), null) - .setPositiveButton(R.string.ok, null) + .setPositiveButton(android.R.string.ok, null) .setCancelable(false) .show(); return true; |