summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authortnmy-cyngn <tlnu@cyngn.com>2016-06-23 09:36:58 +0530
committerSteve Kondik <steve@cyngn.com>2016-08-28 20:43:41 -0700
commitd4f045aefecb054384432ae3d6da8f14044e820a (patch)
tree4f38c6343e8e1801592074a8670f8d6d88564e1a
parent94f5df26c752aee3dbc156d79c5378c1b15f451e (diff)
downloadandroid_packages_apps_Messaging-d4f045aefecb054384432ae3d6da8f14044e820a.tar.gz
android_packages_apps_Messaging-d4f045aefecb054384432ae3d6da8f14044e820a.tar.bz2
android_packages_apps_Messaging-d4f045aefecb054384432ae3d6da8f14044e820a.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
-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 f41211e..a3f373c 100644
--- a/src/com/android/messaging/ui/conversation/ConversationFragment.java
+++ b/src/com/android/messaging/ui/conversation/ConversationFragment.java
@@ -200,7 +200,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);
}
}