diff options
| author | tnmy-cyngn <tlnu@cyngn.com> | 2016-06-23 09:36:58 +0530 |
|---|---|---|
| committer | Michael Bestas <mkbestas@lineageos.org> | 2018-01-21 21:57:19 +0200 |
| commit | e910faa9818607b1fb19d29c2176e09e9df19abe (patch) | |
| tree | 2351efdbbd59d70186d5afa92c8fd7f063fa6eec | |
| parent | a6c4425c696348ce41f77a32d5584b90edee04d6 (diff) | |
| download | android_packages_apps_Messaging-e910faa9818607b1fb19d29c2176e09e9df19abe.tar.gz android_packages_apps_Messaging-e910faa9818607b1fb19d29c2176e09e9df19abe.tar.bz2 android_packages_apps_Messaging-e910faa9818607b1fb19d29c2176e09e9df19abe.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.
Change-Id: Iee7775a74d22808cb262e8973ca70e5e1f0acfc9
Bug-Id: CYNGNOS-3062, KIPPER-718
| -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 a6a191a..419ad5d 100644 --- a/src/com/android/messaging/ui/conversation/ConversationFragment.java +++ b/src/com/android/messaging/ui/conversation/ConversationFragment.java @@ -191,7 +191,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); } } |