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

lutris.profile: allow more syscalls #6067

Merged
merged 1 commit into from
Nov 24, 2023
Merged

Conversation

nutta-git
Copy link
Contributor

@nutta-git nutta-git commented Oct 24, 2023

Need to whitelist ptrace and clone3 for Ubisoft Connect to work.

journalctl did list process_vm_readv when a game was running, but it
didn't crash the game; see
#6035 (comment)

Fixes #6035.

@kmk3 kmk3 changed the title seccomp blacklist for lutris.profile lutris.profile: allow more syscalls Oct 24, 2023
@rusty-snake
Copy link
Collaborator

OT: Do we allow clone3 with restrict-namespaces?

@kmk3
Copy link
Collaborator

kmk3 commented Oct 25, 2023

OT: Do we allow clone3 with restrict-namespaces?

Related to that, I see that no profile currently allows clone3 in seccomp.

From my notes, there was spam about clone3 (syscall=435) in the audit log
(from some other program) unless restrict-namespaces was ignored.

So maybe seccomp already blocks it, though I can't say for sure.

If not, I think it would be better to deal with this in the source code (fix it
or print a warning).

Need to whitelist `ptrace` and `clone3` for Ubisoft Connect to work.

journalctl did list `process_vm_readv` when a game was running, but it
didn't crash the game.

Fixes netblue30#6035.
@kmk3
Copy link
Collaborator

kmk3 commented Nov 1, 2023

OT: Do we allow clone3 with restrict-namespaces?

(Continued on #6076)

@netblue30 netblue30 merged commit 1c69bfb into netblue30:master Nov 24, 2023
3 checks passed
@netblue30
Copy link
Owner

Merged, thanks!

kmk3 added a commit that referenced this pull request Nov 25, 2023
I accidentally removed the `!` when sorting the arguments in #6067.

This amends commit fbba037 ("lutris.profile: allow more syscalls",
2023-10-24) / PR #6067.
kmk3 added a commit to kmk3/firejail that referenced this pull request Nov 25, 2023
It was disabled on commit df6ea88 ("merges, disable sort.py in profile
checks temporarely, two more private-etc profiles", 2023-02-14).

Currently all profiles are sorted and there are no ongoing `private-etc`
changes, so it should be safe to re-enable.

Note that the script is useful to catch sorting issues not only in
`private-etc` but also in other commands, such as `seccomp`[1] [2].

This is a follow-up to netblue30#6070.

Relates to netblue30#5610.

[1] netblue30#6066 (comment)
[2] netblue30#6067 (comment)
kmk3 added a commit to kmk3/firejail that referenced this pull request Nov 25, 2023
It was disabled on commit df6ea88 ("merges, disable sort.py in profile
checks temporarely, two more private-etc profiles", 2023-02-14).

Currently all profiles are sorted and there are no ongoing `private-etc`
changes, so it should be safe to re-enable.

Note that the script is useful to catch sorting issues not only in
`private-etc` but also in other commands, such as `seccomp`[1] [2].

This is a follow-up to netblue30#6070.

Relates to netblue30#5610.

[1] netblue30#6066 (comment)
[2] netblue30#6067 (comment)
@kmk3 kmk3 mentioned this pull request Nov 25, 2023
kmk3 added a commit to kmk3/firejail that referenced this pull request Nov 26, 2023
It was disabled on commit df6ea88 ("merges, disable sort.py in profile
checks temporarely, two more private-etc profiles", 2023-02-14).

Currently all profiles are sorted and there are no ongoing `private-etc`
changes, so it should be safe to re-enable.

Note that the script is useful to catch sorting issues not only in
`private-etc` but also in other commands, such as `seccomp`[1] [2].

This is a follow-up to netblue30#6070.

Relates to netblue30#5610.

[1] netblue30#6066 (comment)
[2] netblue30#6067 (comment)
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.

Lutris: Error at hooking API
4 participants