commit 3f2cf47caf2cb2eafcf94bf127098e1179b7325e
author Adam Seaton <aseaton@google.com> Fri Aug 26 21:13:20 2016 +0000
committer gitbuildkicker <android-build@google.com> Fri Aug 26 16:16:38 2016 -0700
tree 1271d0a2d3c5754202130cf4880098dbcb3a2414
parent 6f753b39e4fb69280a3ef013e37dc88398975489

Revert "Enforce calling identity before clearing."

This reverts commit 8be3a92eb0b4105a9ed748be5a937ce79145f565.

Change-Id: I10401d57239b868f8e3514f81a0e20486838e29c
(cherry picked from commit b440ceb00fd46c9233723066c680a538067fbf82)

src/com/android/providers/downloads/DownloadProvider.java

diff --git a/src/com/android/providers/downloads/DownloadProvider.java b/src/com/android/providers/downloads/DownloadProvider.java
index 667a81d..d9acc78 100644
--- a/src/com/android/providers/downloads/DownloadProvider.java
+++ b/src/com/android/providers/downloads/DownloadProvider.java
@@ -1232,19 +1232,6 @@
             logVerboseOpenFileInfo(uri, mode);
         }
 
-        // Perform normal query to enforce caller identity access before
-        // clearing it to reach internal-only columns
-        final Cursor probeCursor = query(uri, new String[] {
-                Downloads.Impl._DATA }, null, null, null);
-        try {
-            if ((probeCursor == null) || (probeCursor.getCount() == 0)) {
-                throw new FileNotFoundException(
-                        "No file found for " + uri + " as UID " + Binder.getCallingUid());
-            }
-        } finally {
-            IoUtils.closeQuietly(probeCursor);
-        }
-
         final Cursor cursor = queryCleared(uri, new String[] {
                 Downloads.Impl._DATA, Downloads.Impl.COLUMN_STATUS,
                 Downloads.Impl.COLUMN_DESTINATION, Downloads.Impl.COLUMN_MEDIA_SCANNED }, null,