am d4928835: Cleaned out "isValid" from WifiConfig.
* commit 'd49288352908e8cbc9a001e13df2eb35e78deb1a': Cleaned out "isValid" from WifiConfig.
This commit is contained in:
@@ -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;
|
||||
|
||||
Reference in New Issue
Block a user