Skip to content
Snippets Groups Projects
  • Jeff Vander Stoep's avatar
    89d77187
    Remove unnecessary HAL permissions · 89d77187
    Jeff Vander Stoep authored
    Comments indicate that these permissions are used to access already
    open FDs. However, getattr of a directory is clearly not necessary
    for that, search of system_data_file is already granted to domain
    and following symlinks is clearly not needed for reading an already
    open FD.
    
    Bug: 34980020
    Test: boot marlin. Test drm with google play movies, no related
        denials
    Test: cts-tradefed run cts -m CtsMediaTestCases -t \
        android.media.cts.MediaCasTest
        5/6 tests fail with no related selinux denials. The same 5/6
        also fail in selinux permissive mode.
    Change-Id: Ib4b9a1e18bdc479d656b2d64917bbc0358515525
    89d77187
    History
    Remove unnecessary HAL permissions
    Jeff Vander Stoep authored
    Comments indicate that these permissions are used to access already
    open FDs. However, getattr of a directory is clearly not necessary
    for that, search of system_data_file is already granted to domain
    and following symlinks is clearly not needed for reading an already
    open FD.
    
    Bug: 34980020
    Test: boot marlin. Test drm with google play movies, no related
        denials
    Test: cts-tradefed run cts -m CtsMediaTestCases -t \
        android.media.cts.MediaCasTest
        5/6 tests fail with no related selinux denials. The same 5/6
        also fail in selinux permissive mode.
    Change-Id: Ib4b9a1e18bdc479d656b2d64917bbc0358515525