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.

Sign up for GitHub

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

Jump to bottom

efi: skip Read() calls with zero sizes #25922

Merged
merged 1 commit into from
Jan 3, 2023
Merged

efi: skip Read() calls with zero sizes #25922

merged 1 commit into from
Jan 3, 2023

Conversation

poettering
Copy link
Member

Let's avoid calling Read() with zero-sized buffer, to avoid needless firmware quirkiness.

See: #25911

@poettering
efi: skip Read() calls with zero sizes
2fa0e53 
Let's avoid calling Read() with zero-sized buffer, to avoid needless firmware
quirkiness.

See: systemd#25911
@poettering poettering mentioned this pull request Jan 3, 2023
Closed
@github-actions github-actions bot added the please-review PR is ready for (re-)review by a maintainer label Jan 3, 2023
@poettering poettering added quick-review 🏃‍♂️ good-to-merge/waiting-for-ci 👍 PR is good to merge, but CI hasn't passed at time of review. Please merge if you see CI has passed and removed please-review PR is ready for (re-)review by a maintainer labels Jan 3, 2023
@bluca bluca merged commit fd1fec5 into systemd:main Jan 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@bluca bluca bluca approved these changes

@medhefgo medhefgo medhefgo approved these changes

Assignees
No one assigned
Labels
good-to-merge/waiting-for-ci 👍 PR is good to merge, but CI hasn't passed at time of review. Please merge if you see CI has passed sd-boot/sd-stub/bootctl
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@poettering @bluca @medhefgo