diff options
author | tnmy-cyngn <tlnu@cyngn.com> | 2016-06-23 09:36:58 +0530 |
---|---|---|
committer | Gerrit Code Review <gerrit@cyanogenmod.org> | 2016-07-15 10:39:15 -0700 |
commit | 5a05b2f71f2fe43d94c95002e0a9e9dda669ff25 (patch) | |
tree | 8a36dcfbc9ad3f1c41d34fea3bc26f71db51a933 | |
parent | fcd61798bb509ed7244bbfcb48cea38083ac32cc (diff) | |
download | android_packages_apps_Messaging-5a05b2f71f2fe43d94c95002e0a9e9dda669ff25.tar.gz android_packages_apps_Messaging-5a05b2f71f2fe43d94c95002e0a9e9dda669ff25.tar.bz2 android_packages_apps_Messaging-5a05b2f71f2fe43d94c95002e0a9e9dda669ff25.zip |
Messaging app crashes after a few MMS
In some scenarios, data not bound when SIM state
change local broadcasts are received. See stacktrace
for more info. Protect against/skip call to update()
in such scenarios.
Bug-Id: CYNGNOS-3062, KIPPER-718
Change-Id: If94d789aecda47ebff775155c66b670a10ff7fe4
(cherry picked from commit e45eb2f0db9d96071e1279c8002faf41a13cd4d1)
-rw-r--r-- | src/com/android/messaging/ui/conversation/ConversationFragment.java | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/src/com/android/messaging/ui/conversation/ConversationFragment.java b/src/com/android/messaging/ui/conversation/ConversationFragment.java index 448d709..71068b8 100644 --- a/src/com/android/messaging/ui/conversation/ConversationFragment.java +++ b/src/com/android/messaging/ui/conversation/ConversationFragment.java @@ -201,7 +201,8 @@ public class ConversationFragment extends Fragment implements ConversationDataLi intent.getStringExtra(UIIntents.UI_INTENT_EXTRA_CONVERSATION_SELF_ID); Assert.notNull(conversationId); Assert.notNull(selfId); - if (TextUtils.equals(mBinding.getData().getConversationId(), conversationId)) { + if (isBound() && TextUtils + .equals(mBinding.getData().getConversationId(), conversationId)) { mComposeMessageView.updateConversationSelfIdOnExternalChange(selfId); } } |