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

undo change that limits certificate files to a single file #659

Merged
merged 1 commit into from
May 8, 2024

Conversation

jsetje
Copy link
Collaborator

@jsetje jsetje commented Apr 30, 2024

In the ea0f9df "Allow SbatLevel data from external binary" patch the code that tries to load any certificate file that starts with shim_certificate was changed to only accept shim_certificate.efi which defeats the entire point of reading through the directory. This change reverts that.

In the ea0f9df "Allow SbatLevel data from external binary" patch
the code that tries to load any certificate file that starts with
shim_certificate was changed to only accept shim_certificate.efi
which defeats the entire point of reading through the directory.
This change reverts that.

Signed-off-by: Jan Setje-Eilers <Jan.SetjeEilers@oracle.com>
@jsetje
Copy link
Collaborator Author

jsetje commented Apr 30, 2024

This still needs testing.

@jsetje jsetje marked this pull request as ready for review April 30, 2024 20:25
@jsetje
Copy link
Collaborator Author

jsetje commented Apr 30, 2024

Unsurprisingly testing confirms that this restores the previous behavior and shim will load files named:

shim_certificate_001.efi shim_certificate_002.efi etc.

@jsetje jsetje requested a review from vathpela April 30, 2024 20:28
Copy link
Contributor

@vathpela vathpela left a comment

Choose a reason for hiding this comment

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

lgtm.

@vathpela vathpela merged commit 470a8cd into rhboot:main May 8, 2024
21 checks passed
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.

2 participants