Skip to content
Snippets Groups Projects
Commit 9a53bb0f authored by Andres Morales's avatar Andres Morales Committed by Android (Google) Code Review
Browse files

Merge "[gatekeeperd] fix issue with SW->HW upgrades" into mnc-dev

parents e27ee08a 7f6dcf65
Branches
Tags
No related merge requests found
...@@ -144,7 +144,7 @@ public: ...@@ -144,7 +144,7 @@ public:
const gatekeeper::password_handle_t *handle = const gatekeeper::password_handle_t *handle =
reinterpret_cast<const gatekeeper::password_handle_t *>(current_password_handle); reinterpret_cast<const gatekeeper::password_handle_t *>(current_password_handle);
if (handle != NULL && !handle->hardware_backed) { if (handle != NULL && handle->version != 0 && !handle->hardware_backed) {
// handle is being re-enrolled from a software version. HAL probably won't accept // handle is being re-enrolled from a software version. HAL probably won't accept
// the handle as valid, so we nullify it and enroll from scratch // the handle as valid, so we nullify it and enroll from scratch
current_password_handle = NULL; current_password_handle = NULL;
...@@ -209,7 +209,9 @@ public: ...@@ -209,7 +209,9 @@ public:
if (device) { if (device) {
const gatekeeper::password_handle_t *handle = const gatekeeper::password_handle_t *handle =
reinterpret_cast<const gatekeeper::password_handle_t *>(enrolled_password_handle); reinterpret_cast<const gatekeeper::password_handle_t *>(enrolled_password_handle);
if (handle->hardware_backed) { // handle version 0 does not have hardware backed flag, and thus cannot be upgraded to
// a HAL if there was none before
if (handle->version == 0 || handle->hardware_backed) {
ret = device->verify(device, uid, challenge, ret = device->verify(device, uid, challenge,
enrolled_password_handle, enrolled_password_handle_length, enrolled_password_handle, enrolled_password_handle_length,
provided_password, provided_password_length, auth_token, auth_token_length, provided_password, provided_password_length, auth_token, auth_token_length,
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment