Skip to content

Conversation

@alyssais
Copy link
Member

@alyssais alyssais commented Sep 9, 2025

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