summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authortnmy-cyngn <tlnu@cyngn.com>2016-06-23 09:36:58 +0530
committerMichael Bestas <mkbestas@lineageos.org>2019-12-11 20:10:07 +0200
commit7ee39be7e25e20c39f1602da62a766fe69382fc5 (patch)
treedadf8167b5245cd3cb040d99bde251d02120f166
parent2f0f7aee6432f475e84bb4873e7777542757645e (diff)
downloadpackages_apps_Messaging-7ee39be7e25e20c39f1602da62a766fe69382fc5.tar.gz
packages_apps_Messaging-7ee39be7e25e20c39f1602da62a766fe69382fc5.tar.bz2
packages_apps_Messaging-7ee39be7e25e20c39f1602da62a766fe69382fc5.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.java3
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 fbc6730..722bb9c 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);
}
}