summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
author Eric Biggers <ebiggers@google.com> 2023-05-22 17:41:04 +0000
committer Eric Biggers <ebiggers@google.com> 2023-08-09 04:06:06 +0000
commitaa19b22208e11335fda6efea13402b55bec1921f (patch)
tree9d0ca2889c5a4bc3b5bdda01a0bb3996656031be
parentd3252e4e7669ae18970251378706a224b2fcbd3a (diff)
Log errors in verifyChallengeInternal()
It is unexpected for verifyChallengeInternal() to fail, since the Gatekeeper enrollment of the synthetic password (more specifically, the Gatekeeper password derived from the synthetic password) should always be valid. Therefore, ensure that an error message is logged in all failure cases. Bug: 282947979 Change-Id: Ic8d0134bc4f3e0979bf500ffb5a99f2522251d39 Merged-In: Ic8d0134bc4f3e0979bf500ffb5a99f2522251d39 (cherry picked from commit b60583c3b12864e69f10dc01046d898d85b2083a)
-rw-r--r--services/core/java/com/android/server/locksettings/SyntheticPasswordManager.java2
1 files changed, 2 insertions, 0 deletions
diff --git a/services/core/java/com/android/server/locksettings/SyntheticPasswordManager.java b/services/core/java/com/android/server/locksettings/SyntheticPasswordManager.java
index 3f66fee2f86b..f27b913585fd 100644
--- a/services/core/java/com/android/server/locksettings/SyntheticPasswordManager.java
+++ b/services/core/java/com/android/server/locksettings/SyntheticPasswordManager.java
@@ -1327,8 +1327,10 @@ public class SyntheticPasswordManager {
}
return result;
} else if (responseCode == GateKeeperResponse.RESPONSE_RETRY) {
+ Slog.e(TAG, "Gatekeeper verification of synthetic password failed with RESPONSE_RETRY");
return VerifyCredentialResponse.fromTimeout(response.getTimeout());
} else {
+ Slog.e(TAG, "Gatekeeper verification of synthetic password failed with RESPONSE_ERROR");
return VerifyCredentialResponse.ERROR;
}
}