diff options
| author | Jeff Sharkey <jsharkey@android.com> | 2016-08-09 19:45:12 +0000 |
|---|---|---|
| committer | android-build-merger <android-build-merger@google.com> | 2016-08-09 19:45:12 +0000 |
| commit | 860239d87ee4353b9d1bb0196b733c9c632be3f9 (patch) | |
| tree | 7aa606873a5f4f57cb5d088e37d5f0526c0ad8ef /src | |
| parent | 224c8b5d4e15175c9632b25d8a4f3fecc81cc074 (diff) | |
| parent | b3ce7976f2992c0a9860692bc1f766ff1cc6cd29 (diff) | |
| download | android_packages_providers_DownloadProvider-860239d87ee4353b9d1bb0196b733c9c632be3f9.tar.gz android_packages_providers_DownloadProvider-860239d87ee4353b9d1bb0196b733c9c632be3f9.tar.bz2 android_packages_providers_DownloadProvider-860239d87ee4353b9d1bb0196b733c9c632be3f9.zip | |
Enforce calling identity before clearing. am: 8be3a92eb0 am: ec19fe6485
am: b3ce7976f2
Change-Id: I539882f08289cabaaf49326fb7973d98d0323de2
Diffstat (limited to 'src')
| -rw-r--r-- | src/com/android/providers/downloads/DownloadProvider.java | 13 |
1 files changed, 13 insertions, 0 deletions
diff --git a/src/com/android/providers/downloads/DownloadProvider.java b/src/com/android/providers/downloads/DownloadProvider.java index 2d914c41..d2a9d847 100644 --- a/src/com/android/providers/downloads/DownloadProvider.java +++ b/src/com/android/providers/downloads/DownloadProvider.java @@ -1192,6 +1192,19 @@ public final class DownloadProvider extends ContentProvider { 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, |