Merge "Fix NPE issues seen by CTS" into jb-mr2-dev

This commit is contained in:
Irfan Sheriff
2013-04-19 19:43:46 +00:00
committed by Android (Google) Code Review

View File

@@ -412,10 +412,14 @@ public class WifiEnterpriseConfig implements Parcelable {
* @throws IllegalArgumentException if not a CA certificate * @throws IllegalArgumentException if not a CA certificate
*/ */
public void setCaCertificate(X509Certificate cert) { public void setCaCertificate(X509Certificate cert) {
if (cert.getBasicConstraints() >= 0) { if (cert != null) {
mCaCert = cert; if (cert.getBasicConstraints() >= 0) {
mCaCert = cert;
} else {
throw new IllegalArgumentException("Not a CA certificate");
}
} else { } else {
throw new IllegalArgumentException("Not a CA certificate"); mCaCert = null;
} }
} }
@@ -679,6 +683,7 @@ public class WifiEnterpriseConfig implements Parcelable {
} }
private String removeDoubleQuotes(String string) { private String removeDoubleQuotes(String string) {
if (TextUtils.isEmpty(string)) return "";
int length = string.length(); int length = string.length();
if ((length > 1) && (string.charAt(0) == '"') if ((length > 1) && (string.charAt(0) == '"')
&& (string.charAt(length - 1) == '"')) { && (string.charAt(length - 1) == '"')) {