From 845aa217fcd8fb04320183dd80f7001680dd94be Mon Sep 17 00:00:00 2001 From: Taesu Lee Date: Fri, 5 Jun 2020 14:46:51 +0900 Subject: [PATCH] Fix ConcurrentModificationException ConcurrentModificationException could be happened during iterating if parts are added newly. Test: Select several images on Gallery app and attach them to a message via Sharing Signed-off-by: Taesu Lee Change-Id: Idb96239ef2c05a08fc0727797b97739aa89f12a4 --- .../android/messaging/ui/conversation/ComposeMessageView.java | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/src/com/android/messaging/ui/conversation/ComposeMessageView.java b/src/com/android/messaging/ui/conversation/ComposeMessageView.java index 0f36e9a..c35927e 100644 --- a/src/com/android/messaging/ui/conversation/ComposeMessageView.java +++ b/src/com/android/messaging/ui/conversation/ComposeMessageView.java @@ -72,6 +72,7 @@ import com.android.messaging.util.SafeAsyncTask; import com.android.messaging.util.UiUtils; import com.android.messaging.util.UriUtil; +import java.util.ArrayList; import java.util.Collection; import java.util.List; @@ -683,7 +684,8 @@ public class ComposeMessageView extends LinearLayout final boolean hasWorkingDraft = hasMessageText || hasSubject || mBinding.getData().hasAttachments(); - final List attachments = draftMessageData.getReadOnlyAttachments(); + final List attachments = + new ArrayList(draftMessageData.getReadOnlyAttachments()); if (draftMessageData.getIsMms()) { // MMS case if (draftMessageData.hasAttachments()) { if (hasAttachmentsChanged) {