-
Notifications
You must be signed in to change notification settings - Fork 24
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Support PIN through pinentry + session encryption #2
Comments
@Foxboron I remember Alex Wu mentioned during TPMdev2021 conference that Go-tpm does not support parameter encryption session at the time. Back then, Google primarily used go-tpm for virtual TPMs. Later on, encryption session support was added, but I do not know how complete it is. Would you like me to check with Alex and Jim, perhaps also Christ @ Google about this? Cheers, |
The talk from 2021 is just very old at this point. They gave up on trying to implement everything inside the higher-level abstraction It does support session encryption, as you'd expect. Chris also reviewed my usage of the new API in |
Should also mark this as fixed with 2ab0b32 |
I am happy that you solved it. The tpm-direct interface changed things a lot. Originally, go-tpm was made as something that will provide a mild layer for easy of us and safety, and would never give users the direct access (re 2021 go-tpm goals). Funny how things change over time :) |
There is something weird happening with the session encryption when used together with the signing. Inconsistent attributes apparently?
Need to figure out that one.
The text was updated successfully, but these errors were encountered: