mirror of
https://github.com/netblue30/firejail.git
synced 2026-05-15 14:16:14 -06:00
[GH-ISSUE #2609] vlc whitelist #1656
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#1656
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 @Boruch-Baum on GitHub (Mar 19, 2019).
Original GitHub issue: https://github.com/netblue30/firejail/issues/2609
I may be making a newbie error here, but it seems that the default profile for vlc (and I guess clvc) require two whitelist commands in addition to the noblacklist ones:
What happens without those commands are that settings aren't persistent, and vlc always opens with a nag screen.
In addition, I think most people would want the following:
@Boruch-Baum commented on GitHub (Mar 19, 2019):
Actually, looking at the default profile a second time, ALL the
noblacklistitems there should also have correspondingwhitelistones, no? Like I said, I'm new at this. I tried firejail maybe a year or two ago, and am trying it again now.@Vincent43 commented on GitHub (Mar 19, 2019):
As you can see vlc doesn't enable whitelisting for
$HOMEso everything what isn't blacklisted is allowed. You may harden it additionally with various whitelist rules by adding them to/etc/firejail/vlc.local.@Boruch-Baum commented on GitHub (Mar 19, 2019):
@Vincent43 : Really? What you write doesn't make sense. You want to say that adding a
whitelistrule will provide additional hardening ?!?! It does just the opposite! Also, why are you telling me to add personal rules to/etc/firejail/vlc.local? Shouldn't it be~/.config/vlc.profile? On a more fundamental level, what you're claiming aboutwhitelistandblacklistis obviously wrong; you seem to have the language backwards. You're also not addressing the central point of this issue, which is that necessary configuration files for vlc are not by default available.@Vincent43 commented on GitHub (Mar 19, 2019):
Yes.
Because
~/.config/vlc.profilewill overwrite firejail profile completely while/etc/firejail/vlc.localwill add those rules to existing profile.Unfortunately, the central point of this issue is wrong because configuration files for vlc are available by default.
@Boruch-Baum commented on GitHub (Mar 19, 2019):
@Vincent43
Well, if you want to ignore my report, that's certainly an option for you, but what I am reporting contradicts you.
The documentation says to place an
include /etc/firejail/foo.profileat the beginning of one's personal~/.config/firejail/foo.profile, no?A memorable response that I will relish sharing with ALL my friends...
@curiosity-seeker commented on GitHub (Mar 19, 2019):
Please do so if you really want to make a fool of yourself. Instead of bashing @Vincent43 who knows exactly what he‘s talking about you should have read the Firejail documentation - and you would know that adding a whitelist command to a profile means that the respective application does not have access to anything that is not explicitly whitelisted.
@Vincent43 commented on GitHub (Mar 19, 2019):
I'm sorry but your report in not reproducible in current form. You may have something special in your system config or firejail config which causes this but I don't have the crystal ball to know it beforehand.
That could be an option but certainly using
/etc/firejail/foo.localor~/.config/firejail/foo.local(in newer versions) is more appropriate.What I can say... it's the simplest answer I could provide for your question and it's true. You said that you are new to firejail and I can say that I use it as long as it publicly exists and contributed many patches for it. Please consider this for further discussion.
@Vincent43 commented on GitHub (Mar 26, 2019):
Closing as it's not reproducible and no submitter feedback for a week.