Merge \"Ignore system apps on enforceMeteredApnPolicy().\" into nyc-dev (a3d3da8d) · Commits · e / os / android_frameworks_base

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

+8 −4

Original line number	Diff line number	Diff line
		@@ -30,8 +30,6 @@ import static android.net.NetworkCapabilities.NET_CAPABILITY_NOT_RESTRICTED;
		import static android.net.NetworkCapabilities.NET_CAPABILITY_VALIDATED;
		import static android.net.NetworkPolicyManager.RULE_ALLOW_ALL;
		import static android.net.NetworkPolicyManager.RULE_ALLOW_METERED;
		import static android.net.NetworkPolicyManager.MASK_METERED_NETWORKS;
		import static android.net.NetworkPolicyManager.MASK_ALL_NETWORKS;
		import static android.net.NetworkPolicyManager.RULE_NONE;
		import static android.net.NetworkPolicyManager.RULE_REJECT_ALL;
		import static android.net.NetworkPolicyManager.RULE_REJECT_METERED;
		@@ -921,7 +919,7 @@ public class ConnectivityService extends IConnectivityManager.Stub
		// Networks aren't blocked when ignoring blocked status
		if (ignoreBlocked) return false;
		// Networks are never blocked for system services
		if (uid < Process.FIRST_APPLICATION_UID) return false;
		if (isSystem(uid)) return false;

		final boolean networkMetered;
		final int uidRules;
		@@ -4032,12 +4030,18 @@ public class ConnectivityService extends IConnectivityManager.Stub
		return false;
		}

		private boolean isSystem(int uid) {
		return uid < Process.FIRST_APPLICATION_UID;
		}

		private void enforceMeteredApnPolicy(NetworkCapabilities networkCapabilities) {
		final int uid = Binder.getCallingUid();
		if (isSystem(uid)) {
		return;
		}
		// if UID is restricted, don't allow them to bring up metered APNs
		if (networkCapabilities.hasCapability(NET_CAPABILITY_NOT_METERED) == false) {
		final int uidRules;
		final int uid = Binder.getCallingUid();
		synchronized(mRulesLock) {
		uidRules = mUidRules.get(uid, RULE_ALLOW_ALL);
		}

services/core/java/com/android/server/net/NetworkPolicyManagerService.java

+1 −0

Original line number	Diff line number	Diff line
		@@ -1989,6 +1989,7 @@ public class NetworkPolicyManagerService extends INetworkPolicyManager.Stub {
		}

		private void setRestrictBackgroundLocked(boolean restrictBackground) {
		Slog.d(TAG, "setRestrictBackgroundLocked(): " + restrictBackground);
		final boolean oldRestrictBackground = mRestrictBackground;
		mRestrictBackground = restrictBackground;
		// Must whitelist foreground apps before turning data saver mode on.