diff options
| -rw-r--r-- | core/java/android/net/http/SslError.java | 7 | ||||
| -rw-r--r-- | core/java/android/webkit/SslErrorHandlerImpl.java | 4 |
2 files changed, 7 insertions, 4 deletions
diff --git a/core/java/android/net/http/SslError.java b/core/java/android/net/http/SslError.java index 5998f5f..863304c 100644 --- a/core/java/android/net/http/SslError.java +++ b/core/java/android/net/http/SslError.java @@ -198,7 +198,8 @@ public class SslError { /** * Gets the most severe SSL error in this object's set of errors. - * @return The most severe SSL error. + * Returns -1 if the set is empty. + * @return The most severe SSL error, or -1 if the set is empty. */ public int getPrimaryError() { if (mErrors != 0) { @@ -208,9 +209,11 @@ public class SslError { return error; } } + // mErrors should never be set to an invalid value. + assert false; } - return 0; + return -1; } /** diff --git a/core/java/android/webkit/SslErrorHandlerImpl.java b/core/java/android/webkit/SslErrorHandlerImpl.java index 82cd3e8..b2e4b13 100644 --- a/core/java/android/webkit/SslErrorHandlerImpl.java +++ b/core/java/android/webkit/SslErrorHandlerImpl.java @@ -159,7 +159,7 @@ class SslErrorHandlerImpl extends SslErrorHandler { if (DebugFlags.SSL_ERROR_HANDLER) { assert host != null; - assert primary != 0; + assert primary != -1; } if (mSslPrefTable.containsKey(host) && primary <= mSslPrefTable.getInt(host)) { @@ -260,7 +260,7 @@ class SslErrorHandlerImpl extends SslErrorHandler { if (DebugFlags.SSL_ERROR_HANDLER) { assert host != null; - assert primary != 0; + assert primary != -1; } boolean hasKey = mSslPrefTable.containsKey(host); if (!hasKey || primary > mSslPrefTable.getInt(host)) { |