diff options
author | Ekin Oguz <ekinoguz@google.com> | 2016-12-06 02:55:56 +0000 |
---|---|---|
committer | android-build-merger <android-build-merger@google.com> | 2016-12-06 02:55:56 +0000 |
commit | d21eae6d325bc0e1e7d25a67ae68b841bdf0449d (patch) | |
tree | 3527d909073c48233e5bddfb040127ca16aee9d7 | |
parent | 933e59d184b3234cd53ef3e9571b59daaefd780a (diff) | |
parent | eace8b43e8d1512a205554715ca840324bfbe35f (diff) | |
download | android_packages_apps_UnifiedEmail-d21eae6d325bc0e1e7d25a67ae68b841bdf0449d.tar.gz android_packages_apps_UnifiedEmail-d21eae6d325bc0e1e7d25a67ae68b841bdf0449d.tar.bz2 android_packages_apps_UnifiedEmail-d21eae6d325bc0e1e7d25a67ae68b841bdf0449d.zip |
Don't allow file attachment from /data through GET_CONTENT.
am: eace8b43e8
Change-Id: Ib298494836deaee2ba1aa721daf6f496c2c1ee57
-rw-r--r-- | src/com/android/mail/compose/ComposeActivity.java | 18 |
1 files changed, 7 insertions, 11 deletions
diff --git a/src/com/android/mail/compose/ComposeActivity.java b/src/com/android/mail/compose/ComposeActivity.java index 2dfac99b9..5f125abe6 100644 --- a/src/com/android/mail/compose/ComposeActivity.java +++ b/src/com/android/mail/compose/ComposeActivity.java @@ -2018,19 +2018,15 @@ public class ComposeActivity extends ActionBarActivity if (contentUri == null) { return; } - try { - if (handleSpecialAttachmentUri(contentUri)) { - return; - } + if (handleSpecialAttachmentUri(contentUri)) { + return; + } - addAttachmentAndUpdateView(mAttachmentsView.generateLocalAttachment(contentUri)); - } catch (AttachmentFailureException e) { - LogUtils.e(LOG_TAG, e, "Error adding attachment"); - showErrorToast(getResources().getString( - e.getErrorRes(), - AttachmentUtils.convertToHumanReadableSize( - getApplicationContext(), mAccount.settings.getMaxAttachmentSize()))); + final long size = handleAttachmentUrisFromIntent(Arrays.asList(contentUri)); + if (size > 0) { + mAttachmentsChanged = true; + updateSaveUi(); } } |