Commit ec9a1382 authored by Leo Lai's avatar Leo Lai Committed by Commit Bot

InstallAttributes don't check `TpmIsBeingOwned()` to clear owner auth.

We are obsoleting `TpmIsBeingOwned()` of cryptohome API.

BUG=b:172775884
TEST=chromeos_unittests.

Change-Id: Ia6752e3c06ccce3c7c78ebdda2cbe281204a30df
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2556881Reviewed-by: default avatarMaksim Ivanov <emaxx@chromium.org>
Commit-Queue: Leo Lai <cylai@google.com>
Cr-Commit-Position: refs/heads/master@{#830870}
parent 64e0dbe3
...@@ -297,8 +297,7 @@ void InstallAttributes::LockDeviceIfAttributesIsReady( ...@@ -297,8 +297,7 @@ void InstallAttributes::LockDeviceIfAttributesIsReady(
} }
// Clearing the TPM password seems to be always a good deal. // Clearing the TPM password seems to be always a good deal.
if (tpm_util::TpmIsEnabled() && !tpm_util::TpmIsBeingOwned() && if (tpm_util::TpmIsEnabled() && tpm_util::TpmIsOwned()) {
tpm_util::TpmIsOwned()) {
cryptohome_client_->CallTpmClearStoredPasswordAndBlock(); cryptohome_client_->CallTpmClearStoredPasswordAndBlock();
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment