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

docs: warn about limitations of landlock #6302

Merged
merged 1 commit into from
Apr 11, 2024

Conversation

kmk3
Copy link
Collaborator

@kmk3 kmk3 commented Apr 5, 2024

And mark it as experimental.

Relates to #6078.

@glitsj16
Copy link
Collaborator

glitsj16 commented Apr 9, 2024

@kmk3 Although the Firejail/Landlock status is made much clearer in this PR it might still be nice to note that without having landlock in the lsm=x,y,z kernel parameter (cfr. apparmor) Firejail will ignore landlock commands.

Example bootloader lsm param for good Firejail support: lsm=landlock,lockdown,yama,integrity,apparmor,bpf

@kmk3
Copy link
Collaborator Author

kmk3 commented Apr 10, 2024

Although the Firejail/Landlock status is made much clearer in this PR it
might still be nice to note that without having landlock in the lsm=x,y,z
kernel parameter (cfr. apparmor) Firejail will ignore landlock commands.

Example bootloader lsm param for good Firejail support:
lsm=landlock,lockdown,yama,integrity,apparmor,bpf

Landlock should work by default without needing to enable it.

Is there any distribution that disables it?

Example to check if it works:

$ firejail --quiet --noprofile true
$ firejail --quiet --noprofile --landlock.enforce --landlock.fs.read=/foo true
Cannot start application: Permission denied

@glitsj16
Copy link
Collaborator

Is there any distribution that disables it?

Not that I know of. I wasn't clear enough though, let me try to explain what I mean. If a user enables AppArmor as default security model on every boot via the lsm kernel parameter and landlock isn't part of that param, it does get disabled.

And mark it as experimental.

Relates to netblue30#6078.
@kmk3
Copy link
Collaborator Author

kmk3 commented Apr 11, 2024

If a user enables AppArmor as default security model on every boot via the
lsm kernel parameter and landlock isn't part of that param, it does get
disabled.

I see it now; added an item about it.

@netblue30
Copy link
Owner

all set, thanks!

@netblue30 netblue30 merged commit 27cd032 into netblue30:master Apr 11, 2024
2 checks passed
@kmk3 kmk3 deleted the docs-warn-landlock branch April 12, 2024 21:27
@kmk3 kmk3 added the documentation Issues and pull requests related to the documentation label Apr 12, 2024
kmk3 added a commit that referenced this pull request Apr 25, 2024
kmk3 added a commit to kmk3/firejail that referenced this pull request Sep 10, 2024
Reset the bold right after each command/argument.

Command used to check for issues:

    git grep -E ' \\fR' -- src/man/*.in

Related commits:

* e91b9ff ("Deprecate --nodbus option", 2020-04-07) /
  PR netblue30#3265
* 5a61202 ("rename noautopulse to keep-config-pulse", 2021-05-13) /
  PR netblue30#4278
* d79547c ("docs: warn about limitations of landlock", 2024-03-31) /
  PR netblue30#6302

This is a follow-up to netblue30#6451.

Relates to netblue30#6078.
kmk3 added a commit that referenced this pull request Sep 12, 2024
Reset the bold right after each command/argument.

Command used to check for issues:

    git grep -E ' \\fR' -- src/man/*.in

Related commits:

* e91b9ff ("Deprecate --nodbus option", 2020-04-07) /
  PR #3265
* 5a61202 ("rename noautopulse to keep-config-pulse", 2021-05-13) /
  PR #4278
* d79547c ("docs: warn about limitations of landlock", 2024-03-31) /
  PR #6302

This is a follow-up to #6451.

Relates to #6078.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Issues and pull requests related to the documentation
Projects
Status: Done (on RELNOTES)
Development

Successfully merging this pull request may close these issues.

4 participants