Thanks to visit codestin.com
Credit goes to github.com

Skip to content

vmm: error if landlock can't be enabled - #7335

Merged
rbradford merged 1 commit into
cloud-hypervisor:mainfrom
alyssais:landlock-error
Sep 9, 2025
Merged

rbradford merged 1 commit into
cloud-hypervisor:mainfrom
alyssais:landlock-error

Conversation

@alyssais

@alyssais alyssais commented Sep 9, 2025

Copy link
Copy Markdown
Member

Since the user has to explicitly ask for Landlock to be enabled in Cloud Hypervisor, it's surprising that, even when they've done that, Landlock will silently not be enabled if the kernel doesn't support it.

To prevent accidental absence of a desired security protection, exit with an error if Landlock, or the one feature of it we use in Cloud Hypervisor (file access) is not supported.

@alyssais
alyssais requested a review from a team as a code owner September 9, 2025 13:22
Since the user has to explicitly ask for Landlock to be enabled in
Cloud Hypervisor, it's surprising that, even when they've done that,
Landlock will silently not be enabled if the kernel doesn't support
it.

To prevent accidental absence of a desired security protection, exit
with an error if Landlock, or the one feature of it we use in Cloud
Hypervisor (file access) is not supported.

Signed-off-by: Alyssa Ross <[email protected]>
@rbradford
rbradford added this pull request to the merge queue Sep 9, 2025
Merged via the queue into cloud-hypervisor:main with commit 6cac99b Sep 9, 2025
40 of 41 checks passed
@likebreath likebreath moved this from 🆕 New to ✅ Done in Cloud Hypervisor Roadmap Sep 10, 2025
@likebreath likebreath added the bug-fix Bug fix to include in release notes label Sep 10, 2025
@alyssais
alyssais deleted the landlock-error branch November 20, 2025 21:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug-fix Bug fix to include in release notes

Projects

Status: ✅ Done

Development

Successfully merging this pull request may close these issues.

3 participants