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

Commit 0e0d89c2 authored by Hui Yu's avatar Hui Yu
Browse files

Exempt VPN apps from BG-FGS-launch restriction. 

Exempt apps with OP_ACTIVATE_VPN or OP_ACTIVATE_PLATFORM_VPN.

Bug: 182502553.
Test: atest cts/tests/app/src/android/app/cts/ActivityManagerFgsBgStartTest.java#testFgsStartVpn

Change-Id: I0590a0b69a3081fdcd2c9aefe9c00e0e84d41740
parent 2ea75ab9

apex/jobscheduler/framework/java/android/os/PowerExemptionManager.java

+10 −0
Original line number Diff line number Diff line
@@ -176,6 +176,10 @@ public class PowerExemptionManager { 
    public static final int REASON_APPOP = 66;

    /** @hide */

    public static final int REASON_ACTIVITY_VISIBILITY_GRACE_PERIOD = 67;

    /** @hide */

    public static final int REASON_OP_ACTIVATE_VPN = 68;

    /** @hide */

    public static final int REASON_OP_ACTIVATE_PLATFORM_VPN = 69;



    /* BG-FGS-launch is allowed by temp-allow-list or system-allow-list.

       Reason code for temp and system allow list starts here.
@@ -337,6 +341,8 @@ public class PowerExemptionManager { 
            REASON_ALLOWLISTED_PACKAGE,

            REASON_APPOP,

            REASON_ACTIVITY_VISIBILITY_GRACE_PERIOD,

            REASON_OP_ACTIVATE_VPN,

            REASON_OP_ACTIVATE_PLATFORM_VPN,

            // temp and system allow list reasons.

            REASON_GEOFENCING,

            REASON_PUSH_MESSAGING,
@@ -596,6 +602,10 @@ public class PowerExemptionManager { 
                return "APPOP";

            case REASON_ACTIVITY_VISIBILITY_GRACE_PERIOD:

                return "ACTIVITY_VISIBILITY_GRACE_PERIOD";

            case REASON_OP_ACTIVATE_VPN:

                return "OP_ACTIVATE_VPN";

            case REASON_OP_ACTIVATE_PLATFORM_VPN:

                return "OP_ACTIVATE_PLATFORM_VPN";

            case REASON_GEOFENCING:

                return "GEOFENCING";

            case REASON_PUSH_MESSAGING:

services/core/java/com/android/server/am/ActiveServices.java

+13 −0
Original line number Diff line number Diff line
@@ -24,6 +24,8 @@ import static android.app.ActivityManager.PROCESS_STATE_TOP; 
import static android.content.pm.PackageManager.PERMISSION_GRANTED;

import static android.content.pm.ServiceInfo.FOREGROUND_SERVICE_TYPE_MANIFEST;

import static android.os.PowerExemptionManager.REASON_ACTIVITY_VISIBILITY_GRACE_PERIOD;

import static android.os.PowerExemptionManager.REASON_OP_ACTIVATE_PLATFORM_VPN;

import static android.os.PowerExemptionManager.REASON_OP_ACTIVATE_VPN;

import static android.os.PowerWhitelistManager.REASON_ACTIVITY_STARTER;

import static android.os.PowerWhitelistManager.REASON_ALLOWLISTED_PACKAGE;

import static android.os.PowerWhitelistManager.REASON_BACKGROUND_ACTIVITY_PERMISSION;
@@ -5772,6 +5774,17 @@ public final class ActiveServices { 
            }

        }



        if (ret == REASON_DENIED) {

            final AppOpsManager appOpsManager = mAm.getAppOpsManager();

            if (appOpsManager.checkOpNoThrow(AppOpsManager.OP_ACTIVATE_VPN, callingUid,

                    callingPackage) == AppOpsManager.MODE_ALLOWED) {

                ret = REASON_OP_ACTIVATE_VPN;

            } else if (appOpsManager.checkOpNoThrow(AppOpsManager.OP_ACTIVATE_PLATFORM_VPN,

                    callingUid, callingPackage) == AppOpsManager.MODE_ALLOWED) {

                ret = REASON_OP_ACTIVATE_PLATFORM_VPN;

            }

        }



        return ret;

    }