-
Notifications
You must be signed in to change notification settings - Fork 575
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
profiles: replace private-opt with whitelist & document private-opt issues #6021
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
(Adding a NAK mostly due to the duplication)
Misc: It seems like a good idea to avoid private-opt
for very heavy programs
(like electron-based ones), but I'm not sure about never using it on upstream
profiles (I might expand on this later).
@kmk3 Thanks for your review. I had similar doubts about discarding private-opt across the board. True, for less sizable applications the RAM overhead looks tolerable. I'm just not sure where we should draw the line. IMO it's rather likely that users will have way more than one sandbox running in parallel, and things start to add up. I should have elaborated on my reasoning in this context. When I compared the sandbox gains of having private-opt on top of whitelisting the relevant path(s) under /opt I just couldn't see all that much extra hardening to be honest. Looking forward to your insights/arguments. |
6967039
to
6429a21
Compare
In most profiles. Kept private-opt for enpass (~85MB), mate-dictionary (<20MB), minecraft-launcher (~1.6MB) and ppsspp (~44MB). The only app I couldn't check: xmr-stak.
6429a21
to
f7ae17a
Compare
Sorry for the delay, I couldn't get to this earlier.
👍
Not much insight, just this: Pros:
Cons:
I mostly just wanted to give it some consideration, but indeed it's hard to
Sounds good to me. |
Main force-push changes:
It currently looks good to me. Thoughts on the changes? |
That's quite allright, no worries.
I'm okay with your changes. Like you mentioned, there will always be pros & cons to private-opt and users should be aware of the RAM-ifications. Not much more we can do here IMO. Thanks again for your efforts and opinions! |
private-opt breaks file-copy-limit, use a whitelist instead of draining RAM
cfr. #5307
This PR streamlines the comment and does the relevant whitelisting in /opt.