Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 3b1c2545 authored by Jeff Sharkey's avatar Jeff Sharkey
Browse files

Simple mask of incoming file mode. 

Oops, we need to support MODE_WRITE_ONLY.

Bug: 120914823
Test: atest android.os.storage.cts.StorageManagerTest
Change-Id: I9a55656562c9a2c0c2073234fc272a4da24e2a0f
parent 11697f5b

services/core/java/com/android/server/StorageManagerService.java

+1 −6
Original line number Diff line number Diff line
@@ -25,7 +25,6 @@ import static android.content.pm.PackageManager.MATCH_DIRECT_BOOT_AWARE; 
import static android.content.pm.PackageManager.MATCH_DIRECT_BOOT_UNAWARE;

import static android.content.pm.PackageManager.MATCH_UNINSTALLED_PACKAGES;

import static android.content.pm.PackageManager.PERMISSION_GRANTED;

import static android.os.ParcelFileDescriptor.MODE_READ_ONLY;

import static android.os.ParcelFileDescriptor.MODE_READ_WRITE;

import static android.os.storage.OnObbStateChangeListener.ERROR_ALREADY_MOUNTED;

import static android.os.storage.OnObbStateChangeListener.ERROR_COULD_NOT_MOUNT;
@@ -2876,11 +2875,7 @@ class StorageManagerService extends IStorageManager.Stub 
        Slog.v(TAG, "mountProxyFileDescriptor");



        // We only support a narrow set of incoming mode flags

        if ((mode & MODE_READ_WRITE) == MODE_READ_WRITE) {

            mode = MODE_READ_WRITE;

        } else {

            mode = MODE_READ_ONLY;

        }

        mode &= MODE_READ_WRITE;



        try {

            synchronized (mAppFuseLock) {