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

Commit 8058b46a authored by Brad Ebinger's avatar Brad Ebinger
Browse files

Ensure self-managed apps do not need CALL_PHONE to call 

Fixing a regression caused by ag/21803375 which caused
CALL_PHONE to be required for an app implementing a
self-managed CS to pass permission checks.

Also adds some unit tests to ensure that we are starting the
call with the correct params.

Bug: 274524370
Test: CtsTelephonyTestCases:SelfManagedConnectionServiceTest
Change-Id: Ifbc784692e0f00cc60257a06f1fd2a9c6017eb55
parent cb13818e

src/com/android/server/telecom/components/UserCallIntentProcessor.java

+2 −1
Original line number Diff line number Diff line
@@ -140,7 +140,8 @@ public class UserCallIntentProcessor { 
            }

        }



        if (!canCallNonEmergency && !TelephonyUtil.shouldProcessAsEmergency(mContext, handle)) {

        if (!isSelfManaged && !canCallNonEmergency &&

                !TelephonyUtil.shouldProcessAsEmergency(mContext, handle)) {

            showErrorDialogForRestrictedOutgoingCall(mContext,

                    R.string.outgoing_call_not_allowed_no_permission);

            Log.w(this, "Rejecting non-emergency phone call because "

tests/src/com/android/server/telecom/tests/TelecomServiceImplTest.java

+398 −5

File changed.

Preview size limit exceeded, changes collapsed.