From e0f37bb8e119dc45f2444dad501e6538d1699dd8 Mon Sep 17 00:00:00 2001 From: Tom Taylor Date: Thu, 23 Feb 2017 10:41:40 -0800 Subject: [PATCH] 22851197 Messaging Application crash with unsupported type's URI This bug and AOSP patch arrived last July. Supposedly, I merged the change, but it's certainly not fixed in bugler, nor in various copies of PduPersister in Android's master. Fixing all the locations now. Test: manual - sent/received MMS's to verify the change didn't break existing behavior. Bug: 22851197 Change-Id: Ia79bd9951b1820ebc42baf0eef38667753d77ab9 --- src/com/android/messaging/mmslib/pdu/PduPersister.java | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/com/android/messaging/mmslib/pdu/PduPersister.java b/src/com/android/messaging/mmslib/pdu/PduPersister.java index ceb6a85..22578b1 100644 --- a/src/com/android/messaging/mmslib/pdu/PduPersister.java +++ b/src/com/android/messaging/mmslib/pdu/PduPersister.java @@ -1248,7 +1248,8 @@ public class PduPersister { for (int i = 0; i < partsNum; i++) { final PduPart part = body.getPart(i); final Uri partUri = part.getDataUri(); - if ((partUri == null) || !partUri.getAuthority().startsWith("mms")) { + if ((partUri == null) || TextUtils.isEmpty(partUri.getAuthority()) + || !partUri.getAuthority().startsWith("mms")) { toBeCreated.add(part); } else { toBeUpdated.put(partUri, part);