diff options
author | Jay Shrauner <shrauner@google.com> | 2014-01-15 12:11:37 -0800 |
---|---|---|
committer | Jay Shrauner <shrauner@google.com> | 2014-03-01 19:34:20 +0000 |
commit | 2ba6dc6f8bd65176f42491a013c6f1d1d3d4d152 (patch) | |
tree | e4c0362766f52b9cd9d796e8812d69fd3c3d6532 | |
parent | 49a9c120b0b6e3942bc950143f1e2970c8ff374a (diff) | |
download | Contacts-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.java | 15 |
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; + } } } } |