mirror of
https://github.com/netblue30/firejail.git
synced 2026-05-15 14:16:14 -06:00
[GH-ISSUE #571] PulseAudio support is broken when client.conf is a link #403
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#403
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 @j1r1k on GitHub (Jun 14, 2016).
Original GitHub issue: https://github.com/netblue30/firejail/issues/571
Currently there is a check in
firejail/pulseaudio.cwhether/etc/pulse/client.confis a symbolic link (see https://github.com/netblue30/firejail/blob/master/src/firejail/pulseaudio.c#L107).This breaks support for pulseaudio in firejail in NixOS (as client.conf is always symbolic link there).
It seems that removing this check fixes the problem (and pulseaudio works in firejail).
Is there any reason to keep it?
@netblue30 commented on GitHub (Jun 14, 2016):
You are right, there shouldn't be any reason for symlink checking there. I put a fix in git. Thanks for the bug.
@j1r1k commented on GitHub (Jun 14, 2016):
@netblue30 Thanks a lot for a prompt fix.
@netblue30 commented on GitHub (Jun 14, 2016):
No problem.