Skip to content

boot: check if boot assets are trusted by UEFI#16856

Draft
zyga wants to merge 3 commits intocanonical:masterfrom
zyga:feature/check-if-boot-asserts-are-trusted
Draft

boot: check if boot assets are trusted by UEFI#16856
zyga wants to merge 3 commits intocanonical:masterfrom
zyga:feature/check-if-boot-asserts-are-trusted

Conversation

@zyga
Copy link
Copy Markdown
Contributor

@zyga zyga commented Apr 1, 2026

This should prevent writing boot shim that is only signed by the new
Microsoft key, on a machine that doesn't have that key in their UEFI DB.
By the nature of the check we also prevent writing things that have
entries in DBX that would equally fail to boot.

This requires canonical/secboot#532 and will
not pass tests on GitHub.

Signed-off-by: Zygmunt Krynicki zygmunt.krynicki@canonical.com

zyga added 3 commits April 1, 2026 18:48
Signed-off-by: Zygmunt Krynicki <zygmunt.krynicki@canonical.com>
Signed-off-by: Zygmunt Krynicki <zygmunt.krynicki@canonical.com>
This should prevent writing boot shim that is only signed by the new
Microsoft key, on a machine that doesn't have that key in their UEFI DB.
By the nature of the check we also prevent writing things that have
entries in DBX that would equally fail to boot.

This requires canonical/secboot#532

Signed-off-by: Zygmunt Krynicki <zygmunt.krynicki@canonical.com>
@github-actions github-actions Bot added the Run only one system Only runs spread tests on one system label Apr 1, 2026
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 1, 2026

Wed Apr 1 17:16:39 UTC 2026
The following results are from: https://github.com/canonical/snapd/actions/runs/23861135840

No spread failures reported

@pedronis pedronis added the Needs Samuele review Needs a review from Samuele before it can land label Apr 2, 2026
@pedronis
Copy link
Copy Markdown
Contributor

pedronis commented Apr 2, 2026

once this is in a good shape I should give it a look

@alfonsosanchezbeato alfonsosanchezbeato self-requested a review April 2, 2026 12:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Needs Samuele review Needs a review from Samuele before it can land Run only one system Only runs spread tests on one system

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants