Skip to content
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

Readability and Usage #11

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

rvandermate
Copy link

After struggling to understand what to do, and encountering some errors along the way, I've updated the instructions and scripts in the following ways:

  • Made all the scripts executable (had a failure with the service file because it couldn't execute tpm2PolicyConfig)
  • Added PCR 8 to the seal, based on some information I found online that's the GRUB, kernel, and boot commandlines. Note: I didn't have issues like PCR 8 dependency unreliable #5, even after a dist-upgrade after sealing against fresh, un-updated Ubuntu install - although I can see the risk.
  • Updated README.md with step-by-step instructions
  • Updated script with information regarding which password/passphrase is being requested (I didn't know, for instance, that one of the passwords I was entering was for the MOK Enrollment)
  • Removed the service file, and instead instructed to run tpm2PolicyConfig directly after conditions have been met. I could be wrong on this, but I had to do some work turning Secure Boot off for Step 1, then back on after Step 1 - but before Step 2. Also seemed to help with clarity about what was going on, and if it was successful
  • Taking the persistent-handle right from the output of tpm2_evictcontrol (while still printing to the terminal), this fixes The correct reference isn't always the last one #8 (which I also encountered)

rvandermate and others added 5 commits March 1, 2023 12:25
… incorrect, now getting it right from the ouptut of tpm2_evictcontrol
…ted. Removed tpm2keyunlock service, user needs to enable secure boot before executing tpm2PolicyConfig
remove PCR8 (kernel cmdline?) because it was changing after seal
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.

The correct reference isn't always the last one
2 participants