mirror of
https://github.com/netblue30/firejail.git
synced 2026-05-15 14:16:14 -06:00
[PR #3286] [MERGED] Whitelist runuser common #4709
Labels
No labels
LTS merge
LTS merge
bug
bug
converted-to-discussion
doc-todo
documentation
duplicate
enhancement
file-transfer
firecfg
firejail-in-firejail
firetools
graphics
help wanted
information_old
installation
invalid
modif
moved
needinfo
networking
notabug
notourbug
old-version
overlayfs
packaging
profile-request
pull-request
question
question_old
removal
runtime-permissions
sandbox-ipc
security
stale
wiki
wiki
wontfix
wordpress
workaround
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference: github-starred/firejail#4709
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
📋 Pull Request Information
Original PR: https://github.com/netblue30/firejail/pull/3286
Author: @rusty-snake
Created: 3/19/2020
Status: ✅ Merged
Merged: 3/31/2020
Merged by: @rusty-snake
Base:
master← Head:whitelist-runuser-common📝 Commits (10+)
683c0aeintroduce whitelist-runuser-common.inc76ae9fewru update 109b34afadd wruc to more profiles5064405fixes3b4e7edfixes67eb4bfwruc: hide pulse pid6af310aupdate48ea150remove wruc from all the x11 profiles575e424fixes [skip ci]be767bafix ordering [skip ci]📊 Changes
74 files changed (+116 additions, -12 deletions)
View changed files
📝
RELNOTES(+1 -0)📝
etc/baobab.profile(+2 -0)📝
etc/celluloid.profile(+1 -0)📝
etc/curl.profile(+2 -0)📝
etc/d-feet.profile(+1 -0)📝
etc/dconf-editor.profile(+1 -0)📝
etc/dig.profile(+2 -0)📝
etc/elinks.profile(+2 -0)📝
etc/enchant.profile(+1 -0)📝
etc/eo-common.profile(+1 -0)📝
etc/evince.profile(+1 -0)📝
etc/evolution.profile(+2 -0)📝
etc/feedreader.profile(+1 -0)📝
etc/file-roller.profile(+1 -0)📝
etc/file.profile(+1 -0)📝
etc/filezilla.profile(+2 -0)📝
etc/flameshot.profile(+2 -0)📝
etc/gedit.profile(+1 -0)📝
etc/gfeeds.profile(+1 -0)📝
etc/gitg.profile(+1 -0)...and 54 more files
📄 Description
introduce whitelist-runuser-common.inc
60f822e
If an applications does not need a whitelist it can/should be
nowhitelisted. Example:
nowhitelist ${RUNUSER}/pulse
include whitelist-runuser-common.inc
${RUNUSER}/bus is inaccessible with nodbus regardless of the
whitelist. (as it should)
strange wayland setups with an second wayland-compostior need to
whitelist ${RUNUSER}/wayland-1, ${RUNUSER}/wayland-2 and so on.
some display-manager store there Xauthority file in ${RUNUSER}.
test results with fedora 31:
IMPORTANT: ATM we can only enable this for non-graphical and GTK3
programs because mutter (GNOMEs window-manger) stores the Xauthority
file for Xwayland under /run/user/UID/.mutter-Xwaylandauth.XXXXXX
where XXXXXX is random. Until we have whitelist globbing we can't
whitelist this file. QT/KDE and other toolkits without full wayland
support won't be able to start.
wru update 1
286ae11
todo: rebase and apply to gnome-screenshotdone🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.