From 56bacbcb13093191748551c2cbdc4133ba60ec77 Mon Sep 17 00:00:00 2001 From: Jake Klinker Date: Wed, 6 Oct 2021 21:08:32 +0000 Subject: [PATCH] Convert URI scheme to lower case before checking if it is a file. This fixes an edge case where a "File://" style URI could be passed in but not pass this check, even though the underlying framework recognizes this as a legitimate file. Bug: 197328178 Test: See repro steps on http://b/197328178, was no longer able to repro. Change-Id: Id837744a79d1aca91b41d24207dc9b98f2b5b5e9 (cherry picked from commit 9607c95e7164e7856c45fd9d07eea9b09f9db681) --- src/com/android/messaging/util/UriUtil.java | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/src/com/android/messaging/util/UriUtil.java b/src/com/android/messaging/util/UriUtil.java index d336167..1d6a1be 100644 --- a/src/com/android/messaging/util/UriUtil.java +++ b/src/com/android/messaging/util/UriUtil.java @@ -94,8 +94,12 @@ public class UriUtil { return TextUtils.equals(scheme, ContentResolver.SCHEME_ANDROID_RESOURCE); } + /** Returns whether the given Uri is a file. */ public static boolean isFileUri(final Uri uri) { - return uri != null && TextUtils.equals(uri.getScheme(), ContentResolver.SCHEME_FILE); + return uri != null && + uri.getScheme() != null && + TextUtils.equals(uri.getScheme().toLowerCase(), + ContentResolver.SCHEME_FILE); } /**