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

Commit 5d2bb5b5 authored by Mark Chien's avatar Mark Chien Committed by android-build-merger
Browse files

Merge "Don't allow upstream if entitlement don't know any downstream" am: 161814ff 

am: 538bc0fe

Change-Id: Ibfa3dfd6a187f6926831910cbef920cf122721a1
parents 568f8bff 538bc0fe

services/core/java/com/android/server/connectivity/tethering/EntitlementManager.java

+6 −0
Original line number Diff line number Diff line
@@ -161,6 +161,12 @@ public class EntitlementManager { 
     * Check if cellular upstream is permitted.

     */

    public boolean isCellularUpstreamPermitted() {

        // If provisioning is required and EntitlementManager don't know any downstream,

        // cellular upstream should not be allowed.

        final TetheringConfiguration config = mFetcher.fetchTetheringConfiguration();

        if (mCurrentTethers.size() == 0 && isTetherProvisioningRequired(config)) {

            return false;

        }

        return mCellularUpstreamPermitted;

    }

tests/net/java/com/android/server/connectivity/tethering/EntitlementManagerTest.java

+7 −0
Original line number Diff line number Diff line
@@ -405,6 +405,13 @@ public final class EntitlementManagerTest { 


    }



    @Test

    public void verifyPermissionWhenProvisioningNotStarted() {

        assertTrue(mEnMgr.isCellularUpstreamPermitted());

        setupForRequiredProvisioning();

        assertFalse(mEnMgr.isCellularUpstreamPermitted());

    }



    @Test

    public void testRunTetherProvisioning() {

        setupForRequiredProvisioning();