summaryrefslogtreecommitdiffstats
path: root/src
diff options
context:
space:
mode:
authorJeff Sharkey <jsharkey@android.com>2015-07-21 08:57:24 -0700
committerJeff Sharkey <jsharkey@android.com>2015-07-21 08:57:24 -0700
commit182ee96cece8f8fb2c33f127f34152939040e531 (patch)
tree75d9c7e0eaceb6bfc008c637916164b81510266f /src
parent2b9f137db367e7ca9e930ce80d803bdbc2377611 (diff)
downloadandroid_packages_providers_MediaProvider-182ee96cece8f8fb2c33f127f34152939040e531.tar.gz
android_packages_providers_MediaProvider-182ee96cece8f8fb2c33f127f34152939040e531.tar.bz2
android_packages_providers_MediaProvider-182ee96cece8f8fb2c33f127f34152939040e531.zip
Move permission checking from "other" to "group."
The new storage layout used for runtime permissions uses the "group" ACL for enforcing access, and the "other" ACL is usually masked off completely. Bug: 22525550 Change-Id: I29b48cc3425ccfc8d3ff5ec323c8dc50be104878
Diffstat (limited to 'src')
-rwxr-xr-xsrc/com/android/providers/media/MediaProvider.java4
1 files changed, 2 insertions, 2 deletions
diff --git a/src/com/android/providers/media/MediaProvider.java b/src/com/android/providers/media/MediaProvider.java
index e3e3a036..ab7c412d 100755
--- a/src/com/android/providers/media/MediaProvider.java
+++ b/src/com/android/providers/media/MediaProvider.java
@@ -4742,7 +4742,7 @@ public class MediaProvider extends ContentProvider {
try {
StructStat stat = Os.stat(path);
- int accessBits = OsConstants.S_IROTH;
+ int accessBits = OsConstants.S_IRGRP;
if (OsConstants.S_ISREG(stat.st_mode) &&
((stat.st_mode & accessBits) == accessBits)) {
checkLeadingPathComponentsWorldExecutable(path);
@@ -4760,7 +4760,7 @@ public class MediaProvider extends ContentProvider {
throws FileNotFoundException {
File parent = new File(filePath).getParentFile();
- int accessBits = OsConstants.S_IXOTH;
+ int accessBits = OsConstants.S_IXGRP;
while (parent != null) {
if (! parent.exists()) {