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

Commit c710ffbb authored by Treehugger Robot's avatar Treehugger Robot Committed by Automerger Merge Worker
Browse files

Merge "Prevent double-closure of VirtualDeviceImpl" into udc-dev am: f08a7020 am: dc43bbc9 

Original change: https://googleplex-android-review.googlesource.com/c/platform/frameworks/base/+/22746709



Change-Id: I7f0e60bfd1fa23931f80d7f950af99e920138d85
Signed-off-by: default avatarAutomerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
parents ad04fd0f dc43bbc9

services/companion/java/com/android/server/companion/virtual/VirtualDeviceImpl.java

+32 −27
Original line number Diff line number Diff line
@@ -405,9 +405,16 @@ final class VirtualDeviceImpl extends IVirtualDevice.Stub 
    public void close() {

        super.close_enforcePermission();

        // Remove about-to-be-closed virtual device from the service before butchering it.

        mService.removeVirtualDevice(mDeviceId);

        boolean removed = mService.removeVirtualDevice(mDeviceId);

        mDeviceId = Context.DEVICE_ID_INVALID;



        // Device is already closed.

        if (!removed) {

            return;

        }



        final long ident = Binder.clearCallingIdentity();

        try {

            VirtualDisplayWrapper[] virtualDisplaysToBeReleased;

            synchronized (mVirtualDeviceLock) {

                if (mVirtualAudioController != null) {
@@ -428,16 +435,14 @@ final class VirtualDeviceImpl extends IVirtualDevice.Stub 
                mDisplayManager.releaseVirtualDisplay(virtualDisplayWrapper.getToken());

                // The releaseVirtualDisplay call above won't trigger

                // VirtualDeviceImpl.onVirtualDisplayRemoved callback because we already removed the

            // virtual device from the service - we release the other display-tied resources here

            // with the guarantee it will be done exactly once.

                // virtual device from the service - we release the other display-tied resources

                // here with the guarantee it will be done exactly once.

                releaseOwnedVirtualDisplayResources(virtualDisplayWrapper);

            }



            mAppToken.unlinkToDeath(this, 0);

            mCameraAccessController.stopObservingIfNeeded();



        final long ident = Binder.clearCallingIdentity();

        try {

            mInputController.close();

            mSensorController.close();

        } finally {

services/companion/java/com/android/server/companion/virtual/VirtualDeviceManagerService.java

+13 −2
Original line number Diff line number Diff line
@@ -202,8 +202,19 @@ public class VirtualDeviceManagerService extends SystemService { 
        }

    }



    void removeVirtualDevice(int deviceId) {

    /**

     * Remove the virtual device. Sends the

     * {@link VirtualDeviceManager#ACTION_VIRTUAL_DEVICE_REMOVED} broadcast as a result.

     *

     * @param deviceId deviceId to be removed

     * @return {@code true} if the device was removed, {@code false} if the operation was a no-op

     */

    boolean removeVirtualDevice(int deviceId) {

        synchronized (mVirtualDeviceManagerLock) {

            if (!mVirtualDevices.contains(deviceId)) {

                return false;

            }



            mAppsOnVirtualDevices.remove(deviceId);

            mVirtualDevices.remove(deviceId);

        }
@@ -223,6 +234,7 @@ public class VirtualDeviceManagerService extends SystemService { 
        } finally {

            Binder.restoreCallingIdentity(identity);

        }

        return true;

    }



    private void syncVirtualDevicesToCdmAssociations(List<AssociationInfo> associations) {
@@ -248,7 +260,6 @@ public class VirtualDeviceManagerService extends SystemService { 
        for (VirtualDeviceImpl virtualDevice : virtualDevicesToRemove) {

            virtualDevice.close();

        }



    }



    private void registerCdmAssociationListener() {