summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorJeff Sharkey <jsharkey@android.com>2016-03-21 17:12:39 -0600
committerJeff Sharkey <jsharkey@android.com>2016-03-21 17:12:52 -0600
commita013d5b1a3a66297ea9471767fcbfef7f6ba4d69 (patch)
tree3624a684925f856c67695e7488ee5c1a2898c20a
parent5f46b22b67c83305bc330e9c24f6afaeba05a832 (diff)
downloadandroid_packages_providers_DownloadProvider-a013d5b1a3a66297ea9471767fcbfef7f6ba4d69.zip
android_packages_providers_DownloadProvider-a013d5b1a3a66297ea9471767fcbfef7f6ba4d69.tar.gz
android_packages_providers_DownloadProvider-a013d5b1a3a66297ea9471767fcbfef7f6ba4d69.tar.bz2
Remove chmod(0644) for finished downloads.
Now that PackageInstaller can read APKs from content:// Uris, we don't need to make downloads world-readable. This is mostly just cleanup, since our top-level private data directory is no longer o+x, so other apps can't traverse in anyway. Bug: 5464052 Change-Id: I45de6a40c28b85c64d5afbc13068fe3ae8341d93
-rw-r--r--src/com/android/providers/downloads/DownloadThread.java6
1 files changed, 0 insertions, 6 deletions
diff --git a/src/com/android/providers/downloads/DownloadThread.java b/src/com/android/providers/downloads/DownloadThread.java
index 325b4ee..65142db 100644
--- a/src/com/android/providers/downloads/DownloadThread.java
+++ b/src/com/android/providers/downloads/DownloadThread.java
@@ -624,12 +624,6 @@ public class DownloadThread implements Runnable {
} else if (Downloads.Impl.isStatusSuccess(mInfoDelta.mStatus)) {
// When success, open access if local file
if (mInfoDelta.mFileName != null) {
- try {
- // TODO: remove this once PackageInstaller works with content://
- Os.chmod(mInfoDelta.mFileName, 0644);
- } catch (ErrnoException ignored) {
- }
-
if (mInfo.mDestination != Downloads.Impl.DESTINATION_FILE_URI) {
try {
// Move into final resting place, if needed