-
-
Notifications
You must be signed in to change notification settings - Fork 580
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
initial pledge(2) on OpenBSD #1803
Conversation
Limit the set of system calls shairport-sync is allowed to make, most importantly fork(2)/execve(2) if and only if user defined commands are run. OpenBSD's official audio/shairport-sync port already ships with this patch. https://man.openbsd.org/pledge.2
Here's what already works and has already been tested by other OpenBSD users.
|
I've also tested this with AirPlay 2 Support and mikebrady/nqptp#31, i.e. shared memory between the two unprivileged and pledged programs does work and sound is played. |
Thanks again for all this work! I’ll try it out with the (patched) NQPTP. |
If it helps, https://github.com/jasperla/openbsd-wip/tree/master/net/nqptp is a ready-to-use NQPTP port incl. this PR,
Then tweak https://github.com/openbsd/ports/tree/master/audio/shairport-sync to build with AirPlay 2 and do
|
Friendly ping. |
Oops — I think I misunderstood, and was leaving the PRs open. Please let me know what you’d like me to do — accept the two PRs, or what. Thanks. |
This PR
It is ready for review/merge -- the other remains a draft I can work with later on. |
Many thanks for this! |
Limit the set of system calls shairport-sync is allowed to make, most importantly fork(2)/execve(2) if and only if user defined commands are run.
OpenBSD's official audio/shairport-sync port already ships with this patch.
https://man.openbsd.org/pledge.2