mirror of
https://github.com/netblue30/firejail.git
synced 2026-05-15 14:16:14 -06:00
[GH-ISSUE #5378] Yet another --private-etc symlink issue, with /etc/alternatives/ #2976
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#2976
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?
Originally created by @JulienPalard on GitHub (Sep 23, 2022).
Original GitHub issue: https://github.com/netblue30/firejail/issues/5378
Description
Depending on how I set --private-etc,
/etc/alternatives/ccis sometimes a symlink, sometimes a copy:Steps to Reproduce
If I whitelist
/etc/alternatives/ccalone, I get a copy ofccinstead of its symlink:Which is bad (cc is not happy to be here). On the other hand, if I just whielist the whole
alternativesdirectory:this time
/etc/alternatives/ccis a symlink (andccis happy). Butfcopywhines about a 500MB limit being reached, which is probably wrong as /etc/alternatives only contains symlinks:Expected behavior
I know that symlink is a security issue magnet, so I may miss something obvious, but as there's already a way to have the symlink I quickly though "what about the other way also giving a symlink ?".
Environment
Checklist
/usr/bin/vlc) "fixes" it).https://github.com/netblue30/firejail/issues/1139)browser-allow-drm yes/browser-disable-u2f noinfirejail.configto allow DRM/U2F in browsers.--profile=PROFILENAMEto set the right profile. (Only relevant for AppImages)@gerasiov commented on GitHub (Aug 13, 2023):
The problem is in the logic used for size calculation: for symlinks fcopy looks at target file, not at the symlink itself.
@ghost commented on GitHub (Aug 13, 2023):
Yet another...
FYI, there's been quite a few PR's marked as drafts due to the private-etc refactoring. Most importantly IMO is #5645. Somewhere along the line this should be decided upon. IMO best before the next release.Just my $ 0.02