Skip to content

uefi runtime: Increase default size of name buffer #1579

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

Merged
merged 4 commits into from
Mar 21, 2025

Conversation

ptf2
Copy link
Contributor

@ptf2 ptf2 commented Mar 20, 2025

Some firmware fails to update the "VariableNameSize" parameter passed into GetNextVariableName(). To work around this we will use a large default buffer size.

@ptf2 ptf2 force-pushed the enlarge-default-name-buffer branch 2 times, most recently from 63ded99 to 7385a2f Compare March 20, 2025 21:33
@nicholasbishop
Copy link
Member

Add a bullet on this change to uefi/CHANGELOG.md file.

"VariableKeys now uses a larger internal buffer to work around firmware bugs on some devices.", or something like that.

Copy link
Member

@phip1611 phip1611 left a comment

Choose a reason for hiding this comment

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

Please add a changelog entry, as nicholas pointed out

ptf2 added 4 commits March 21, 2025 14:48
Some firmware fails to update the "VariableNameSize" parameter passed
into GetNextVariableName(). To work around this we will use a large
default buffer size.
Address PR comments: update code comments, move `use alloc::vec` into
appropriate cfg block.
Describe VariableKeys name buffer change.
@phip1611 phip1611 force-pushed the enlarge-default-name-buffer branch from 204e007 to b43d34c Compare March 21, 2025 13:48
@nicholasbishop nicholasbishop added this pull request to the merge queue Mar 21, 2025
Merged via the queue into rust-osdev:main with commit 6fed32e Mar 21, 2025
16 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.

3 participants