commit 68974f3e9592f58c2802cb33e44cf6bd5e0105f2
author Yi Chou <yich@google.com> Mon Oct 26 11:48:23 2020 +0800
committer Commit Bot <commit-bot@chromium.org> Sat Oct 31 11:21:14 2020 +0000
tree 0e59efde09877fb7f7098fd031519567231229d8
parent a328c5bf1445b4c1f3635da5edf58ea0c1019ef1

cryptohome: remove redundant tpm_init call

The ownership callback would connect at PostDBusInitialize, and SetupTpm
would call by crypto_->Init(tpm_init_).
We didn't need to do those thing again.

BUG=chromium:879797
TEST=FEATURES=test emerge-soraka cryptohome
TEST=tast run $DUT hwsec.Cryptohome*
TEST=Manually enroll, login, logout on soraka

Change-Id: I51e04224fdb9155b40db6499bb79f8851eaa2ae7
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform2/+/2497374
Tested-by: joe Chou <yich@google.com>
Commit-Queue: joe Chou <yich@google.com>
Reviewed-by: Leo Lai <cylai@google.com>

cryptohome/userdataauth.cc

diff --git a/cryptohome/userdataauth.cc b/cryptohome/userdataauth.cc
index 26a82d8..55b4fa0 100644
--- a/cryptohome/userdataauth.cc
+++ b/cryptohome/userdataauth.cc
@@ -286,9 +286,6 @@
   // We expect |tpm_| and |tpm_init_| to be available by this point.
   DCHECK(tpm_ && tpm_init_);
 
-  tpm_init_->Init(
-      base::Bind(&UserDataAuth::OwnershipCallback, base::Unretained(this)));
-
   // Seed /dev/urandom
   SeedUrandom();