Skip to content
This repository has been archived by the owner on Jun 27, 2018. It is now read-only.

Add restrictive seccomp whitelist #285

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

DemiMarie
Copy link

This adds a restrictive seccomp whitelist. This was a major security loss when Playpen was replaced by Docker.

This requires that whitelist.json be placed in the working directory of the process. Not sure how to arrange for that. That file is generated from whitelist by convert.py. I used the same whitelist as was used back when Playpen was the sandbox.

@alexcrichton
Copy link
Member

Can this be integrated into how the server is compiled and/or executes docker? As is it seems difficult to deploy and not well integrated.

@DemiMarie
Copy link
Author

My understanding is that Docker expects the JSON file to actually exist on the filesystem, but that file could be created at runtime.

@alexcrichton
Copy link
Member

Yes creating the file at runtime would be ok

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants