sailfishos / sailjail-permissions

Other
5 stars 17 forks source link

Add booster profile files #79

Closed spiiroin closed 3 years ago

spiiroin commented 3 years ago

Boosting sandboxed applications works by launching booster in sandbox. Such application boosters use permissions for the the application they are handling. Any additional sandboxing tweaks that are needed are handled by use of booster type specific profile files.

Signed-off-by: Simo Piiroinen simo.piiroinen@jolla.com

Tomin1 commented 3 years ago

Just a thought, but should we have these profile files for applications separate from profile files for boosters? I application binaries and booster binaries are stored separately (/usr/bin vs. /usr/libexec/mapplaunched). Then again it might not be a real issue as apps should not be prefixed with "booster-".

I assume these booster permissions don't get dropped when the app is executed, so does this open some holes like launching apps from a sandbox? That would basically mean that until that's fixed one should be careful which sandboxed apps get boosted.

spiiroin commented 3 years ago

This looks fine as long as we have a plan to take care of any possible holes this might introduce.

As it is not possible to modify sandbox once it is running, applications started via boosted sandbox will always have a bit wider view to system than the same app without boosting. That just needs to be taken in account when deciding whether boosting is done or not along with other details such as increased memory use etc. That said:

spiiroin commented 3 years ago

Rebased

spiiroin commented 3 years ago

Review commits squashed