diff options
| author | Parménides GV <parmegv@sdf.org> | 2013-12-10 17:39:43 +0100 |
|---|---|---|
| committer | Parménides GV <parmegv@sdf.org> | 2013-12-10 17:39:43 +0100 |
| commit | dca94d79c3391568b4228cab26ea2f02de46cde4 (patch) | |
| tree | 8d69ad83da319ea4d9443714d760430a96f26c37 /src | |
| parent | ee87387c8318796986e7e5dd4fb1278eab7f9624 (diff) | |
Setting up a provider w/o connection doesn't fail.
There was a nullpointer already fixed in another branch which
completely complements this one. The message shown to the user is empty,
but that belongs to bug/more-detailed-response-to-CW-errors.
Diffstat (limited to 'src')
| -rw-r--r-- | src/se/leap/bitmaskclient/ProviderAPI.java | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/se/leap/bitmaskclient/ProviderAPI.java b/src/se/leap/bitmaskclient/ProviderAPI.java index b23bdc7f..2a1cd29b 100644 --- a/src/se/leap/bitmaskclient/ProviderAPI.java +++ b/src/se/leap/bitmaskclient/ProviderAPI.java @@ -448,7 +448,7 @@ public class ProviderAPI extends IntentService { private boolean validCertificate(String cert_string) { boolean result = false; - if(!cert_string.isEmpty()) { + if(!ConfigHelper.checkErroneousDownload(cert_string)) { X509Certificate certCert = ConfigHelper.parseX509CertificateFromString(cert_string); try { Base64.encodeToString( certCert.getEncoded(), Base64.DEFAULT); |