summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
author Rubin Xu <rubinxu@google.com> 2017-02-20 11:03:22 +0000
committer Android (Google) Code Review <android-gerrit@google.com> 2017-02-20 11:03:26 +0000
commit82c2d94e89c0d786b0734ff98a8395e1a7ce3956 (patch)
tree5ce96828235ca266c221c6dceb698094ef8f6a76
parent7bde762f3beeb69aab73b8af4fa554f81c6e2ff9 (diff)
parentdf406d802e9f6348f307ebbe1809c5cdc6c0b64e (diff)
Merge "Normalize empty credential into null"
-rw-r--r--services/core/java/com/android/server/LockSettingsService.java8
1 files changed, 8 insertions, 0 deletions
diff --git a/services/core/java/com/android/server/LockSettingsService.java b/services/core/java/com/android/server/LockSettingsService.java
index 4a4453046295..f76ddc71bc01 100644
--- a/services/core/java/com/android/server/LockSettingsService.java
+++ b/services/core/java/com/android/server/LockSettingsService.java
@@ -1064,6 +1064,14 @@ public class LockSettingsService extends ILockSettings.Stub {
private void setLockCredentialInternal(String credential, int credentialType,
String savedCredential, int userId) throws RemoteException {
+ // Normalize savedCredential and credential such that empty string is always represented
+ // as null.
+ if (TextUtils.isEmpty(savedCredential)) {
+ savedCredential = null;
+ }
+ if (TextUtils.isEmpty(credential)) {
+ credential = null;
+ }
synchronized (mSpManager) {
if (isSyntheticPasswordBasedCredentialLocked(userId)) {
spBasedSetLockCredentialInternalLocked(credential, credentialType, savedCredential,