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

tool: Add error for write locked #450

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

crawfxrd
Copy link
Member

Add a new error for the case of trying to flash when security is enabled and it is still locked and update the related docs.

@crawfxrd crawfxrd requested review from jackpot51 and a team March 22, 2024 21:00
@crawfxrd crawfxrd marked this pull request as ready for review March 22, 2024 21:01
Add a new error for the case of trying to flash when security is enabled
and it is still locked and update the related docs.

Signed-off-by: Tim Crawford <tcrawford@system76.com>
@crawfxrd crawfxrd changed the title Add error for write locked tool: Add error for write locked Mar 22, 2024
@XV-02 XV-02 self-assigned this Apr 15, 2024
@XV-02
Copy link

XV-02 commented Apr 17, 2024

After running ectool.sh security unlock I get presented with the unlock prompt on every boot - hot or cold - until I cancel out of it.

That doesn't feel correct? I would expect it to prompt for a single boot, and then the standard locked behaviour to reassert on subsequent boots.

@XV-02
Copy link

XV-02 commented Apr 17, 2024

Also, should the post ectool.sh security unlock message highlight that a reboot is insufficient to correctly change the security state?

Currently tool/src/main.rs line 299 prints "Shut down the system for the security state to take effect" which is technically correct, but could perhaps more forcibly state that a cold boot is required?

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.

None yet

3 participants