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 9607c95e71)
This commit is contained in:
Jake Klinker
2021-10-06 21:08:32 +00:00
parent 7f113e2a81
commit 56bacbcb13

View File

@@ -94,8 +94,12 @@ public class UriUtil {
return TextUtils.equals(scheme, ContentResolver.SCHEME_ANDROID_RESOURCE); return TextUtils.equals(scheme, ContentResolver.SCHEME_ANDROID_RESOURCE);
} }
/** Returns whether the given Uri is a file. */
public static boolean isFileUri(final Uri uri) { 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);
} }
/** /**