Cleaned out "isValid" from WifiConfig. (d4928835) · Commits · e / os / android_frameworks_base

wifi/java/android/net/wifi/WifiConfiguration.java

+2 −56

Original line number	Diff line number	Diff line
		@@ -918,60 +918,6 @@ public class WifiConfiguration implements Parcelable {
		creatorUid = -1;
		}

		/**
		* indicates whether the configuration is valid
		* @return true if valid, false otherwise
		* @hide
		*/
		public boolean isValid() {
		String reason = strIsValid();
		if (reason != null) {
		Log.e(TAG, "WiFi Config not valid: " + reason);
		return false;
		}
		else {
		return true;
		}
		}

		private String strIsValid() {

		if (allowedKeyManagement == null)
		return "allowed kmgmt";

		if (allowedKeyManagement.cardinality() > 1) {
		if (allowedKeyManagement.cardinality() != 2) {
		return "cardinality != 2";
		}
		if (!allowedKeyManagement.get(KeyMgmt.WPA_EAP)) {
		return "not WPA_EAP";
		}
		if ((!allowedKeyManagement.get(KeyMgmt.IEEE8021X))
		&& (!allowedKeyManagement.get(KeyMgmt.WPA_PSK))) {
		return "not PSK or 8021X";
		}
		}

		if (!TextUtils.isEmpty(FQDN)) {
		/* this is passpoint configuration; it must not have an SSID */
		if (!TextUtils.isEmpty(SSID)) {
		return "SSID not expected for Passpoint: '" + SSID + "'";
		}
		/* this is passpoint configuration; it must have a providerFriendlyName */
		if (TextUtils.isEmpty(providerFriendlyName)) {
		return "no provider friendly name";
		}
		/* this is passpoint configuration; it must have enterprise config */
		if (enterpriseConfig == null
		\|\| enterpriseConfig.getEapMethod() == WifiEnterpriseConfig.Eap.NONE ) {
		return "no enterprise config";
		}
		}

		// TODO: Add more checks
		return null;
		}

		/**
		* Identify if this configuration represents a passpoint network
		*/
		@@ -1327,8 +1273,8 @@ public class WifiConfiguration implements Parcelable {

		/** @hide */
		public int getAuthType() {
		if (isValid() == false) {
		throw new IllegalStateException("Invalid configuration");
		if (allowedKeyManagement.cardinality() > 1) {
		throw new IllegalStateException("More than one auth type set");
		}
		if (allowedKeyManagement.get(KeyMgmt.WPA_PSK)) {
		return KeyMgmt.WPA_PSK;