summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorJay Shrauner <shrauner@google.com>2014-01-15 12:11:37 -0800
committerJay Shrauner <shrauner@google.com>2014-03-01 19:34:20 +0000
commit2ba6dc6f8bd65176f42491a013c6f1d1d3d4d152 (patch)
treee4c0362766f52b9cd9d796e8812d69fd3c3d6532
parent49a9c120b0b6e3942bc950143f1e2970c8ff374a (diff)
downloadContacts-2ba6dc6f8bd65176f42491a013c6f1d1d3d4d152.tar.gz
Fix NPE in bindEditor
Correctly handle null return values from getMimeEntries() Bug:12568010 Change-Id: Ic43afb302fa0bdf7852b2247745eaefffa9f7492 (cherry picked from commit ea1b1c62c3232f89d9a56450abf0003af434a962)
-rw-r--r--src/com/android/contacts/activities/ConfirmAddDetailActivity.java15
1 files changed, 9 insertions, 6 deletions
diff --git a/src/com/android/contacts/activities/ConfirmAddDetailActivity.java b/src/com/android/contacts/activities/ConfirmAddDetailActivity.java
index d60cc7385..3f9116fa1 100644
--- a/src/com/android/contacts/activities/ConfirmAddDetailActivity.java
+++ b/src/com/android/contacts/activities/ConfirmAddDetailActivity.java
@@ -683,12 +683,15 @@ public class ConfirmAddDetailActivity extends Activity implements
// Skip kind that are not editable
if (!kind.editable) continue;
if (mMimetype.equals(kind.mimeType)) {
- for (ValuesDelta valuesDelta : mRawContactDelta.getMimeEntries(mMimetype)) {
- // Skip entries that aren't visible
- if (!valuesDelta.isVisible()) continue;
- if (valuesDelta.isInsert()) {
- inflateEditorView(kind, valuesDelta, mRawContactDelta);
- return;
+ final ArrayList<ValuesDelta> deltas = mRawContactDelta.getMimeEntries(mMimetype);
+ if (deltas != null) {
+ for (ValuesDelta valuesDelta : deltas) {
+ // Skip entries that aren't visible
+ if (!valuesDelta.isVisible()) continue;
+ if (valuesDelta.isInsert()) {
+ inflateEditorView(kind, valuesDelta, mRawContactDelta);
+ return;
+ }
}
}
}