diff options
| author | 2023-05-22 17:41:04 +0000 | |
|---|---|---|
| committer | 2023-08-09 04:06:06 +0000 | |
| commit | aa19b22208e11335fda6efea13402b55bec1921f (patch) | |
| tree | 9d0ca2889c5a4bc3b5bdda01a0bb3996656031be | |
| parent | d3252e4e7669ae18970251378706a224b2fcbd3a (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.java | 2 |
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; } } |