Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@josephlr
Copy link
Member

@josephlr josephlr commented Dec 22, 2018

For certain TPM2 commands like ActivateCredential or PolicySecret, the
TPM is allowed to return TPM_RC_RETRY. In this simulator, this occurs
when dictionary attack prevention mechanisms are used. Specifically,
when checking to see if the maximum number of tries has been exceeded,
the simulator returns TPM_RC_RETRY to let certain values sync to nvdata.

While this can occur on real TPMs as well, it tends not to, and go-tpm
does not automatically handle this case
. So we simply remove the above
feature.

For certain TPM2 commands like ActivateCredential or PolicySecret, the
TPM is allowed to return TPM_RC_RETRY. In this simulator, this occurs
when dictionary attack prevention mechanisms are used. Specifically,
when checking to see if the maximum number of tries has been exceeded,
the simulator returns TPM_RC_RETRY to let certain values sync to nvdata.

While this can occur on real TPMs as well, it tends not to, and go-tpm
does not automatically handle this case. So we simply remove the above
feature.
Copy link

@twitchy-jsonp twitchy-jsonp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

Nothing really to review here.

@josephlr josephlr merged commit 94e14ab into master Dec 22, 2018
@josephlr josephlr deleted the use_da_used branch December 22, 2018 01:39
josephlr pushed a commit to josephlr/go-tpm-tools that referenced this pull request Jul 3, 2019
taking latest from parent repo
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants