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

Commit affb2296 authored by Edward Savage-Jones's avatar Edward Savage-Jones Committed by Lorenzo Colitti
Browse files

Add NETWORK_AIRPLANE_MODE permission 

Add a permission to allow airplane mode to
be toggled without holding the NETWORK_SETTINGS
(or similar) permission.

Bug: 141621384
Bug: 145164696
Test: TODO
Change-Id: Iebe2f501fdd802d54fd3345ac2fb0a8c129faf71
parent 43429e77

api/system-current.txt

+2 −1
Original line number Diff line number Diff line
@@ -125,6 +125,7 @@ package android { 
    field public static final String MODIFY_PARENTAL_CONTROLS = "android.permission.MODIFY_PARENTAL_CONTROLS";

    field public static final String MODIFY_QUIET_MODE = "android.permission.MODIFY_QUIET_MODE";

    field public static final String MOVE_PACKAGE = "android.permission.MOVE_PACKAGE";

    field public static final String NETWORK_AIRPLANE_MODE = "android.permission.NETWORK_AIRPLANE_MODE";

    field public static final String NETWORK_CARRIER_PROVISIONING = "android.permission.NETWORK_CARRIER_PROVISIONING";

    field public static final String NETWORK_MANAGED_PROVISIONING = "android.permission.NETWORK_MANAGED_PROVISIONING";

    field public static final String NETWORK_SCAN = "android.permission.NETWORK_SCAN";
@@ -4343,7 +4344,7 @@ package android.net { 
    method @RequiresPermission(android.Manifest.permission.TETHER_PRIVILEGED) public void getLatestTetheringEntitlementResult(int, boolean, @NonNull java.util.concurrent.Executor, @NonNull android.net.ConnectivityManager.OnTetheringEntitlementResultListener);

    method @RequiresPermission(anyOf={android.Manifest.permission.TETHER_PRIVILEGED, android.Manifest.permission.WRITE_SETTINGS}) public boolean isTetheringSupported();

    method @RequiresPermission(android.Manifest.permission.TETHER_PRIVILEGED) public void registerTetheringEventCallback(@NonNull java.util.concurrent.Executor, @NonNull android.net.ConnectivityManager.OnTetheringEventCallback);

    method @RequiresPermission(anyOf={android.Manifest.permission.NETWORK_SETTINGS, android.Manifest.permission.NETWORK_SETUP_WIZARD, android.Manifest.permission.NETWORK_STACK}) public void setAirplaneMode(boolean);

    method @RequiresPermission(anyOf={android.Manifest.permission.NETWORK_AIRPLANE_MODE, android.Manifest.permission.NETWORK_SETTINGS, android.Manifest.permission.NETWORK_SETUP_WIZARD, android.Manifest.permission.NETWORK_STACK}) public void setAirplaneMode(boolean);

    method @RequiresPermission(anyOf={android.net.NetworkStack.PERMISSION_MAINLINE_NETWORK_STACK, android.Manifest.permission.NETWORK_STACK}) public boolean shouldAvoidBadWifi();

    method @RequiresPermission(android.net.NetworkStack.PERMISSION_MAINLINE_NETWORK_STACK) public void startCaptivePortalApp(@NonNull android.net.Network, @NonNull android.os.Bundle);

    method @RequiresPermission(android.Manifest.permission.TETHER_PRIVILEGED) public void startTethering(int, boolean, android.net.ConnectivityManager.OnStartTetheringCallback);

core/java/android/net/ConnectivityManager.java

+1 −0
Original line number Diff line number Diff line
@@ -3155,6 +3155,7 @@ public class ConnectivityManager { 
     * @hide

     */

    @RequiresPermission(anyOf = {

            android.Manifest.permission.NETWORK_AIRPLANE_MODE,

            android.Manifest.permission.NETWORK_SETTINGS,

            android.Manifest.permission.NETWORK_SETUP_WIZARD,

            android.Manifest.permission.NETWORK_STACK})

core/res/AndroidManifest.xml

+6 −0
Original line number Diff line number Diff line
@@ -1601,6 +1601,12 @@ 
    <permission android:name="android.permission.REQUEST_NETWORK_SCORES"

        android:protectionLevel="signature|setup" />



    <!-- @SystemApi @hide Allows applications to toggle airplane mode.

         <p>Not for use by third-party or privileged applications.

    -->

    <permission android:name="android.permission.NETWORK_AIRPLANE_MODE"

        android:protectionLevel="signature" />



    <!-- Allows network stack services (Connectivity and Wifi) to coordinate

         <p>Not for use by third-party or privileged applications.

         @SystemApi

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

+10 −1
Original line number Diff line number Diff line
@@ -2060,6 +2060,15 @@ public class ConnectivityService extends IConnectivityManager.Stub 
                NetworkStack.PERMISSION_MAINLINE_NETWORK_STACK);

    }



    private void enforceAirplaneModePermission() {

        enforceAnyPermissionOf(

                android.Manifest.permission.NETWORK_AIRPLANE_MODE,

                android.Manifest.permission.NETWORK_SETTINGS,

                android.Manifest.permission.NETWORK_SETUP_WIZARD,

                android.Manifest.permission.NETWORK_STACK,

                NetworkStack.PERMISSION_MAINLINE_NETWORK_STACK);

    }



    private boolean checkNetworkStackPermission() {

        return checkAnyPermissionOf(

                android.Manifest.permission.NETWORK_STACK,
@@ -4900,7 +4909,7 @@ public class ConnectivityService extends IConnectivityManager.Stub 


    @Override

    public void setAirplaneMode(boolean enable) {

        enforceNetworkStackSettingsOrSetup();

        enforceAirplaneModePermission();

        final long ident = Binder.clearCallingIdentity();

        try {

            final ContentResolver cr = mContext.getContentResolver();