Skip to content

profiles: seccomp: update to Linux 5.11 syscall list#4957

Merged
AkihiroSuda merged 1 commit intocontainerd:masterfrom
thaJeztah:update_seccomp_profile
Jan 22, 2021
Merged

profiles: seccomp: update to Linux 5.11 syscall list#4957
AkihiroSuda merged 1 commit intocontainerd:masterfrom
thaJeztah:update_seccomp_profile

Conversation

@thaJeztah
Copy link
Copy Markdown
Member

This applies the same changes as were made by @cyphar in moby/moby#41889


These syscalls (some of which have been in Linux for a while but were
missing from the profile) fall into a few buckets:

  • close_range(2), epoll_wait2(2) are just extensions of existing "safe
    for everyone" syscalls.

  • The mountv2 API syscalls (fs*(2), move_mount(2), open_tree(2)) are
    all equivalent to aspects of mount(2) and thus go into the
    CAP_SYS_ADMIN category.

  • process_madvise(2) is similar to the other process_*(2) syscalls and
    thus goes in the CAP_SYS_PTRACE category.

Co-authored-by: Aleksa Sarai [email protected]
Signed-off-by: Sebastiaan van Stijn [email protected]

These syscalls (some of which have been in Linux for a while but were
missing from the profile) fall into a few buckets:

 * close_range(2), epoll_wait2(2) are just extensions of existing "safe
   for everyone" syscalls.

 * The mountv2 API syscalls (fs*(2), move_mount(2), open_tree(2)) are
   all equivalent to aspects of mount(2) and thus go into the
   CAP_SYS_ADMIN category.

 * process_madvise(2) is similar to the other process_*(2) syscalls and
   thus goes in the CAP_SYS_PTRACE category.

Co-authored-by: Aleksa Sarai <[email protected]>
Signed-off-by: Sebastiaan van Stijn <[email protected]>
@theopenlab-ci
Copy link
Copy Markdown

theopenlab-ci Bot commented Jan 21, 2021

Build succeeded.

Copy link
Copy Markdown
Member

@estesp estesp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Copy Markdown
Contributor

@Zyqsempai Zyqsempai left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@AkihiroSuda AkihiroSuda merged commit 0bbbc59 into containerd:master Jan 22, 2021
@thaJeztah thaJeztah deleted the update_seccomp_profile branch January 22, 2021 09:43
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.

4 participants