Slightly simplify IpManager#compareProvisioning. am: b77b775213 (6f8180cf) · Commits · e / os / android_frameworks_base

services/net/java/android/net/ip/IpManager.java

+6 −4

Original line number	Diff line number	Diff line
		@@ -701,6 +701,10 @@ public class IpManager extends StateMachine {
		delta = ProvisioningChange.LOST_PROVISIONING;
		}

		final boolean lostIPv6 = oldLp.isIPv6Provisioned() && !newLp.isIPv6Provisioned();
		final boolean lostIPv4Address = oldLp.hasIPv4Address() && !newLp.hasIPv4Address();
		final boolean lostIPv6Router = oldLp.hasIPv6DefaultRoute() && !newLp.hasIPv6DefaultRoute();

		// Additionally:
		//
		// Partial configurations (e.g., only an IPv4 address with no DNS
		@@ -713,8 +717,7 @@ public class IpManager extends StateMachine {
		// Because on such a network isProvisioned() will always return false,
		// delta will never be LOST_PROVISIONING. So check for loss of
		// provisioning here too.
		if ((oldLp.hasIPv4Address() && !newLp.hasIPv4Address()) \|\|
		(oldLp.isIPv6Provisioned() && !newLp.isIPv6Provisioned())) {
		if (lostIPv4Address \|\| lostIPv6) {
		delta = ProvisioningChange.LOST_PROVISIONING;
		}

		@@ -723,8 +726,7 @@ public class IpManager extends StateMachine {
		// If the previous link properties had a global IPv6 address and an
		// IPv6 default route then also consider the loss of that default route
		// to be a loss of provisioning. See b/27962810.
		if (oldLp.hasGlobalIPv6Address() && oldLp.hasIPv6DefaultRoute() &&
		!newLp.hasIPv6DefaultRoute()) {
		if (oldLp.hasGlobalIPv6Address() && lostIPv6Router) {
		delta = ProvisioningChange.LOST_PROVISIONING;
		}