diff options
author | Jeff Sharkey <jsharkey@android.com> | 2016-03-21 17:12:39 -0600 |
---|---|---|
committer | Jeff Sharkey <jsharkey@android.com> | 2016-03-21 17:12:52 -0600 |
commit | a013d5b1a3a66297ea9471767fcbfef7f6ba4d69 (patch) | |
tree | 3624a684925f856c67695e7488ee5c1a2898c20a /src/com/android | |
parent | 5f46b22b67c83305bc330e9c24f6afaeba05a832 (diff) | |
download | android_packages_providers_DownloadProvider-a013d5b1a3a66297ea9471767fcbfef7f6ba4d69.tar.gz android_packages_providers_DownloadProvider-a013d5b1a3a66297ea9471767fcbfef7f6ba4d69.tar.bz2 android_packages_providers_DownloadProvider-a013d5b1a3a66297ea9471767fcbfef7f6ba4d69.zip |
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
Diffstat (limited to 'src/com/android')
-rw-r--r-- | src/com/android/providers/downloads/DownloadThread.java | 6 |
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 325b4eee..65142db6 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 |