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

Commit 9099a061 authored by Nate Myren's avatar Nate Myren
Browse files

Ensure that SOFT_DENIED doesn't result in a call to finishOp 

Bug: 190385491
Test: atest SensorPrivacyMicrophoneTest
Change-Id: Ib739d9fe0d9aae46199f6e7f3c2f4e0f51396b51
parent 1de80b1d

services/core/java/com/android/server/pm/permission/PermissionManagerService.java

+2 −1
Original line number Diff line number Diff line
@@ -5495,7 +5495,8 @@ public class PermissionManagerService extends IPermissionManager.Stub { 
            final int result = checkPermission(mContext, permission, attributionSource, message,

                    forDataDelivery, startDataDelivery, fromDatasource, attributedOp);

            // Finish any started op if some step in the attribution chain failed.

            if (startDataDelivery && result != PermissionChecker.PERMISSION_GRANTED) {

            if (startDataDelivery && result != PermissionChecker.PERMISSION_GRANTED

                    && result != PermissionChecker.PERMISSION_SOFT_DENIED) {

                if (attributedOp == AppOpsManager.OP_NONE) {

                    finishDataDelivery(AppOpsManager.permissionToOpCode(permission),

                            attributionSource.asState(), fromDatasource);