diff options
author | Sudheer Shanka <sudheersai@google.com> | 2019-04-30 23:52:33 +0000 |
---|---|---|
committer | Android (Google) Code Review <android-gerrit@google.com> | 2019-04-30 23:52:33 +0000 |
commit | 7cff85ee863cbd69ec3856dcde71809f29607eb4 (patch) | |
tree | b93df5f19368cda58f37e53c066715c0580f8f89 | |
parent | 178b6d89e9799473679a0d50417bda8bae2ec963 (diff) | |
parent | fce964a60cb69041d47d4bf4f37ab08232bde86a (diff) | |
download | android_packages_providers_DownloadProvider-7cff85ee863cbd69ec3856dcde71809f29607eb4.tar.gz android_packages_providers_DownloadProvider-7cff85ee863cbd69ec3856dcde71809f29607eb4.tar.bz2 android_packages_providers_DownloadProvider-7cff85ee863cbd69ec3856dcde71809f29607eb4.zip |
Merge "Skip restrictions if DownloadProvider itself is trying to add a download." into qt-dev
-rw-r--r-- | src/com/android/providers/downloads/DownloadProvider.java | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/src/com/android/providers/downloads/DownloadProvider.java b/src/com/android/providers/downloads/DownloadProvider.java index a7f2093c..4a95361d 100644 --- a/src/com/android/providers/downloads/DownloadProvider.java +++ b/src/com/android/providers/downloads/DownloadProvider.java @@ -1111,7 +1111,9 @@ public final class DownloadProvider extends ContentProvider { final boolean runningLegacyMode = appOpsManager.checkOp(AppOpsManager.OP_LEGACY_STORAGE, Binder.getCallingUid(), getCallingPackage()) == AppOpsManager.MODE_ALLOWED; - if (Helpers.isFilenameValidInExternalPackage(getContext(), file, getCallingPackage())) { + if (Binder.getCallingPid() == Process.myPid()) { + return; + } else if (Helpers.isFilenameValidInExternalPackage(getContext(), file, getCallingPackage())) { // No permissions required for paths belonging to calling package. return; } else if ((runningLegacyMode && Helpers.isFilenameValidInPublicDownloadsDir(file)) |