New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 665596 link

Starred by 0 users

Issue metadata

Status: Archived
Owner: ----
Closed: Nov 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 3
Type: Bug



Sign in to add a comment

UnitTest failure on gru-paladin

Project Member Reported by skau@chromium.org, Nov 15 2016

Issue description

tpm_manager-0.0.1-r596: [WARNING:tpm2_initializer_impl.cc(136)] WARNING: Verified boot PCR 0 is not initialized.
tpm_manager-0.0.1-r596: [WARNING:tpm2_initializer_impl.cc(136)] WARNING: Verified boot PCR 1 is not initialized.
tpm_manager-0.0.1-r596: [WARNING:tpm2_initializer_impl.cc(136)] WARNING: Verified boot PCR 2 is not initialized.
tpm_manager-0.0.1-r596: [WARNING:tpm2_initializer_impl.cc(136)] WARNING: Verified boot PCR 3 is not initialized.
tpm_manager-0.0.1-r596: ../../../../../../../tmp/portage/chromeos-base/tpm_manager-0.0.1-r596/work/tpm_manager-0.0.1/aosp/system/tpm/tpm_manager/server/tpm2_initializer_test.cc:148: Failure
tpm_manager-0.0.1-r596: Actual function call count doesn't match EXPECT_CALL(mock_tpm_utility_, ExtendPCR(_, _, _))...
tpm_manager-0.0.1-r596:          Expected: to be called at least 4 times
tpm_manager-0.0.1-r596:            Actual: never called - unsatisfied and active
tpm_manager-0.0.1-r596: [  FAILED  ] Tpm2InitializerTest.PCRSpoofGuard (29 ms)
 
Ah, right, need to update the unit tests as well :(

Comment 3 by skau@chromium.org, Nov 15 2016

This CL looks to be the culprit
https://chromium-review.googlesource.com/#/c/411514/

Comment 5 by skau@chromium.org, Nov 16 2016

Status: Fixed (was: Untriaged)
Expect next CQ run to be green.  Author of CL fixed it.

Comment 6 by dchan@google.com, Mar 4 2017

Labels: VerifyIn-58

Comment 7 by dchan@google.com, Apr 17 2017

Labels: VerifyIn-59

Comment 8 by dchan@google.com, May 30 2017

Labels: VerifyIn-60

Comment 9 by dchan@chromium.org, Aug 1 2017

Labels: VerifyIn-61

Comment 10 by dchan@chromium.org, Oct 14 2017

Status: Archived (was: Fixed)

Sign in to add a comment