diff options
author | David Smith <davidas@google.com> | 2018-05-01 14:54:46 -0700 |
---|---|---|
committer | Atanas Kirilov <akirilov@google.com> | 2018-05-25 23:24:02 +0000 |
commit | 9deefdf182d69f98d361d9d24de1e4892923dac5 (patch) | |
tree | 73d1207f3852fd9c2b03c6593dde5e041178f60c /src/com/android/messaging/ui | |
parent | 6ea30b87b14ca682c6cae2ab2e3f121a88d4121b (diff) | |
download | packages_apps_Messaging-9deefdf182d69f98d361d9d24de1e4892923dac5.tar.gz packages_apps_Messaging-9deefdf182d69f98d361d9d24de1e4892923dac5.tar.bz2 packages_apps_Messaging-9deefdf182d69f98d361d9d24de1e4892923dac5.zip |
Messaging ignores file URIs shared via intent
Fixes b/37629504 by not creating attachments from file URIs shared via
intent. Sharing file URIs is disallowed by Android N and later and is no
longer supported for this app.
Test: manual using the Debug command added in this change
Change-Id: Ibff486c94ac703a9a4c5a80e33b934a460804a8f
Merged-In: Ibff486c94ac703a9a4c5a80e33b934a460804a8f
Diffstat (limited to 'src/com/android/messaging/ui')
-rw-r--r-- | src/com/android/messaging/ui/conversationlist/ShareIntentActivity.java | 13 |
1 files changed, 13 insertions, 0 deletions
diff --git a/src/com/android/messaging/ui/conversationlist/ShareIntentActivity.java b/src/com/android/messaging/ui/conversationlist/ShareIntentActivity.java index 83b7be9..1c91e46 100644 --- a/src/com/android/messaging/ui/conversationlist/ShareIntentActivity.java +++ b/src/com/android/messaging/ui/conversationlist/ShareIntentActivity.java @@ -35,6 +35,7 @@ import com.android.messaging.util.ContentType; import com.android.messaging.util.LogUtil; import com.android.messaging.util.MediaMetadataRetrieverWrapper; import com.android.messaging.util.FileUtil; +import com.android.messaging.util.UriUtil; import java.io.IOException; import java.util.ArrayList; @@ -75,6 +76,12 @@ public class ShareIntentActivity extends BaseBugleActivity implements final String action = intent.getAction(); if (Intent.ACTION_SEND.equals(action)) { final Uri contentUri = (Uri) intent.getParcelableExtra(Intent.EXTRA_STREAM); + if (UriUtil.isFileUri(contentUri)) { + LogUtil.i( + LogUtil.BUGLE_TAG, + "Ignoring attachment from file URI which are no longer supported."); + return; + } final String contentType = extractContentType(contentUri, intent.getType()); if (LogUtil.isLoggable(LogUtil.BUGLE_TAG, LogUtil.DEBUG)) { LogUtil.d(LogUtil.BUGLE_TAG, String.format( @@ -112,6 +119,12 @@ public class ShareIntentActivity extends BaseBugleActivity implements if (imageUris != null && imageUris.size() > 0) { mDraftMessage = MessageData.createSharedMessage(null); for (final Uri imageUri : imageUris) { + if (UriUtil.isFileUri(imageUri)) { + LogUtil.i( + LogUtil.BUGLE_TAG, + "Ignoring attachment from file URI which are no longer supported."); + continue; + } final String actualContentType = extractContentType(imageUri, contentType); addSharedImagePartToDraft(actualContentType, imageUri); } |