Android/frameworks_base
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Cleaned out "isValid" from WifiConfig.

Browse Source 
Change-Id: I028615e55700aa92889670297f1130397eba650f
(cherry picked from commit c50315e014)
This commit is contained in:
Jan Nordqvist
2015-06-01 16:14:55 -07:00
committed by Vinit Deshpande
parent 597492c963
commit d492883529
1 changed files with 2 additions and 56 deletions
Download Patch File Download Diff File Expand all files Collapse all files

58
wifi/java/android/net/wifi/WifiConfiguration.java
View File

@@ -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;