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

Commit f863543d authored by Lorenzo Colitti's avatar Lorenzo Colitti Committed by Automerger Merge Worker
Browse files

Clear calling identity in registerNetworkAgent. am: feda5907 

Original change: https://android-review.googlesource.com/c/platform/frameworks/base/+/1511317

Change-Id: I0f93630527cfe352f5d8f5c69b5276dadbb8c3e4
parents aa3254e8 feda5907

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

+22 −11
Original line number Diff line number Diff line
@@ -5980,13 +5980,29 @@ public class ConnectivityService extends IConnectivityManager.Stub 
            int currentScore, NetworkAgentConfig networkAgentConfig, int providerId) {

        if (networkCapabilities.hasTransport(TRANSPORT_TEST)) {

            enforceAnyPermissionOf(Manifest.permission.MANAGE_TEST_NETWORKS);

        } else {

            enforceNetworkFactoryPermission();

        }



        final int uid = Binder.getCallingUid();

        final long token = Binder.clearCallingIdentity();

        try {

            return registerNetworkAgentInternal(messenger, networkInfo, linkProperties,

                    networkCapabilities, currentScore, networkAgentConfig, providerId, uid);

        } finally {

            Binder.restoreCallingIdentity(token);

        }

    }



    private Network registerNetworkAgentInternal(Messenger messenger, NetworkInfo networkInfo,

            LinkProperties linkProperties, NetworkCapabilities networkCapabilities,

            int currentScore, NetworkAgentConfig networkAgentConfig, int providerId, int uid) {

        if (networkCapabilities.hasTransport(TRANSPORT_TEST)) {

            // Strictly, sanitizing here is unnecessary as the capabilities will be sanitized in

            // the call to mixInCapabilities below anyway, but sanitizing here means the NAI never

            // sees capabilities that may be malicious, which might prevent mistakes in the future.

            networkCapabilities = new NetworkCapabilities(networkCapabilities);

            networkCapabilities.restrictCapabilitesForTestNetwork(Binder.getCallingUid());

        } else {

            enforceNetworkFactoryPermission();

            networkCapabilities.restrictCapabilitesForTestNetwork(uid);

        }



        LinkProperties lp = new LinkProperties(linkProperties);
@@ -5997,7 +6013,7 @@ public class ConnectivityService extends IConnectivityManager.Stub 
        final NetworkAgentInfo nai = new NetworkAgentInfo(messenger, new AsyncChannel(),

                new Network(mNetIdManager.reserveNetId()), new NetworkInfo(networkInfo), lp, nc,

                currentScore, mContext, mTrackerHandler, new NetworkAgentConfig(networkAgentConfig),

                this, mNetd, mDnsResolver, mNMS, providerId, Binder.getCallingUid());

                this, mNetd, mDnsResolver, mNMS, providerId, uid);



        // Make sure the LinkProperties and NetworkCapabilities reflect what the agent info says.

        processCapabilitiesFromAgent(nai, nc);
@@ -6008,13 +6024,8 @@ public class ConnectivityService extends IConnectivityManager.Stub 
        final String name = TextUtils.isEmpty(extraInfo)

                ? nai.networkCapabilities.getSsid() : extraInfo;

        if (DBG) log("registerNetworkAgent " + nai);

        final long token = Binder.clearCallingIdentity();

        try {

        mDeps.getNetworkStack().makeNetworkMonitor(

                nai.network, name, new NetworkMonitorCallbacks(nai));

        } finally {

            Binder.restoreCallingIdentity(token);

        }

        // NetworkAgentInfo registration will finish when the NetworkMonitor is created.

        // If the network disconnects or sends any other event before that, messages are deferred by

        // NetworkAgent until nai.asyncChannel.connect(), which will be called when finalizing the