diff options
author | Jeff Sharkey <jsharkey@android.com> | 2016-09-16 18:03:54 -0600 |
---|---|---|
committer | Jeff Sharkey <jsharkey@android.com> | 2016-09-16 18:03:54 -0600 |
commit | e2c5d91b95a39a1e5028581bd72bd0e0ea907caf (patch) | |
tree | 64e8dc4db7b669988fac7d11b1101e600feb5fc2 /src/com | |
parent | e1f211b04e1db0935f3a60eabcc2c4e462420787 (diff) | |
parent | 010fc1856c23d5a15a6e42e334b0fdc7986f7f30 (diff) | |
download | android_packages_providers_DownloadProvider-e2c5d91b95a39a1e5028581bd72bd0e0ea907caf.tar.gz android_packages_providers_DownloadProvider-e2c5d91b95a39a1e5028581bd72bd0e0ea907caf.tar.bz2 android_packages_providers_DownloadProvider-e2c5d91b95a39a1e5028581bd72bd0e0ea907caf.zip |
Merge commit '010fc1856c23d5a15a6e42e334b0fdc7986f7f30' into manual_merge_010fc18
Change-Id: I2fa7bbc82985a294564a072650f9e8472dae9694
Diffstat (limited to 'src/com')
-rw-r--r-- | src/com/android/providers/downloads/DownloadProvider.java | 44 |
1 files changed, 44 insertions, 0 deletions
diff --git a/src/com/android/providers/downloads/DownloadProvider.java b/src/com/android/providers/downloads/DownloadProvider.java index 8fd06d94..b2b2c08d 100644 --- a/src/com/android/providers/downloads/DownloadProvider.java +++ b/src/com/android/providers/downloads/DownloadProvider.java @@ -468,6 +468,19 @@ public final class DownloadProvider extends ContentProvider { if (appInfo != null) { mDefContainerUid = appInfo.uid; } + + // Grant access permissions for all known downloads to the owning apps + final SQLiteDatabase db = mOpenHelper.getReadableDatabase(); + final Cursor cursor = db.query(DB_TABLE, new String[] { + Downloads.Impl._ID, Constants.UID }, null, null, null, null, null); + try { + while (cursor.moveToNext()) { + grantAllDownloadsPermission(cursor.getLong(0), cursor.getInt(1)); + } + } finally { + cursor.close(); + } + return true; } @@ -690,6 +703,7 @@ public final class DownloadProvider extends ContentProvider { } insertRequestHeaders(db, rowID, values); + grantAllDownloadsPermission(rowID, Binder.getCallingUid()); notifyContentChanged(uri, match); final long token = Binder.clearCallingIdentity(); @@ -1216,6 +1230,7 @@ public final class DownloadProvider extends ContentProvider { reader.updateFromDatabase(info); scheduler.cancel((int) info.mId); + revokeAllDownloadsPermission(info.mId); DownloadStorageProvider.onDownloadProviderDelete(getContext(), info.mId); final String path = info.mFileName; @@ -1273,6 +1288,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, @@ -1455,4 +1483,20 @@ public final class DownloadProvider extends ContentProvider { to.put(key, defaultValue); } } + + private void grantAllDownloadsPermission(long id, int uid) { + final String[] packageNames = getContext().getPackageManager().getPackagesForUid(uid); + if (packageNames == null || packageNames.length == 0) return; + + // We only need to grant to the first package, since the + // platform internally tracks based on UIDs + final Uri uri = ContentUris.withAppendedId(Downloads.Impl.ALL_DOWNLOADS_CONTENT_URI, id); + getContext().grantUriPermission(packageNames[0], uri, + Intent.FLAG_GRANT_READ_URI_PERMISSION | Intent.FLAG_GRANT_WRITE_URI_PERMISSION); + } + + private void revokeAllDownloadsPermission(long id) { + final Uri uri = ContentUris.withAppendedId(Downloads.Impl.ALL_DOWNLOADS_CONTENT_URI, id); + getContext().revokeUriPermission(uri, ~0); + } } |