Skip to content

seccomp: whitelist io_pgetevents()#38895

Merged
tonistiigi merged 1 commit intomoby:masterfrom
avikivity:io_pgetevents
Mar 21, 2019
Merged

seccomp: whitelist io_pgetevents()#38895
tonistiigi merged 1 commit intomoby:masterfrom
avikivity:io_pgetevents

Conversation

@avikivity
Copy link
Contributor

io_pgetevents() is a new Linux system call. It is similar to io_getevents()
that is already whitelisted, and adds no special abilities over that system call.

Allow that system call to enable applications that use it.

Fixes #38894.

Signed-off-by: Avi Kivity avi@scylladb.com

@thaJeztah
Copy link
Member

ping @justincormack @jessfraz PTAL

io_pgetevents() is a new Linux system call. It is similar to io_getevents()
that is already whitelisted, and adds no special abilities over that system call.

Allow that system call to enable applications that use it.

Fixes #38894.

Signed-off-by: Avi Kivity <avi@scylladb.com>
@codecov
Copy link

codecov bot commented Mar 18, 2019

Codecov Report

❗ No coverage uploaded for pull request base (master@36d2c8b). Click here to learn what that means.
The diff coverage is 0%.

@@            Coverage Diff            @@
##             master   #38895   +/-   ##
=========================================
  Coverage          ?   36.46%           
=========================================
  Files             ?      614           
  Lines             ?    45877           
  Branches          ?        0           
=========================================
  Hits              ?    16727           
  Misses            ?    26867           
  Partials          ?     2283

@avikivity
Copy link
Contributor Author

I dropped a change in the vendor/ tree, after the automated checks complained, and force-pushed.

Copy link
Member

@thaJeztah thaJeztah left a comment

Choose a reason for hiding this comment

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

LGTM

@tonistiigi tonistiigi merged commit 6daf5ab into moby:master Mar 21, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants