diff options
author | Jeff Sharkey <jsharkey@android.com> | 2015-07-21 08:57:24 -0700 |
---|---|---|
committer | Jeff Sharkey <jsharkey@android.com> | 2015-07-21 08:57:24 -0700 |
commit | 182ee96cece8f8fb2c33f127f34152939040e531 (patch) | |
tree | 75d9c7e0eaceb6bfc008c637916164b81510266f | |
parent | 2b9f137db367e7ca9e930ce80d803bdbc2377611 (diff) | |
download | android_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
-rwxr-xr-x | src/com/android/providers/media/MediaProvider.java | 4 |
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()) { |