-
Notifications
You must be signed in to change notification settings - Fork 569
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
qBittorrent tray icon missing from notification panel when running it with firejail #3707
Comments
run with |
For the record: Tray-icon and native-notifications (there are multiple standards in the linux world, IDK how all of them work and which software implements which):
qBittorent may fail with that becaue it has no own rules and error-handling differs between "permission denied" on socket and "permission denied" inside dbus. A hardened solution for 0.9.62:
|
I ran This should probably be added to the qbittorrent.profile. |
Hmm, qbittorent is a program which mainly deals with (untrusted) data from the internet. This means it has a high attack-surface (e.g. buffer-overflows while processing a bad packet can result in RCE). D-Bus is a good technology for sandbox-escapes. So just allowing all D-Bus is risky. Fortunately we have fine grained D-Bus access policies since 0.9.64. However a lot of D-Bus interfaces are still unsafe and |
That's why I'm wary of adding it (and I'm not sure what's being allowed when also blacklisting /run/dbus). But somehow qBittorrent and other apps with tray icons should work (completely) with firejail. Looks like Debian recently upgraded its used firejail version in backports to 0.9.64: is it now possible to allow tray-icons in specific? Does this mean that allowing notification area icons would still be unsafe with dbus (if so and if you know related dbus code issues please link them here; haven't looked into zeromq which is probably unlikely to be viable)? |
/run/dbus/system_bus_socket is the socket of the system-bus which is mainly used for privileged things (NetworkManager, systemd, PackageKit, Bluetooth, ...).
Yes, you can install it and try the rules I posted above. OT: I suggest to always use the backports version because it has newer features and the much more recent profiles.
It's not a bug in the D-Bus code. If you want it is a bug in the spec. The issue is that the most DEs populate all there objects/methods/interfaces on all names. Example on GNOME (<= 3.36.1) as I use GNOME and am more familiar but KDE should do the same IIRC. IDK which of the names above are really needed and IDK if they safe under KDE, but based on my previous experience I classify all names as unsafe until I know better. Using EDIT: and there are some reviews: https://github.com/netblue30/firejail/wiki/Restrict-D-Bus |
firejail 0.9.72-1 on Arch Linux on KDE Plasma 5.115.0-1 and qbittorrent 4.6.3-1
Also for future readers like me, link has been moved to https://github.com/netblue30/firejail/wiki/Restrict-DBus . |
The system tray icon of qBittorrent missing when minimizing it to the notification area
Bug and expected behavior
I get this in the console when starting qBittorrent with firejail:
Warning: An abstract unix socket for session D-BUS might still be available. Use --net or remove unix from --protocol set
org.kde.knotifications: env says KDE is running but SNI unavailable -- check KDE_FULL_SESSION and XDG_CURRENT_DESKTOP
No profile and disabling firejail
firejail --noprofile /path/to/program
in a terminal?I don't get these 2 outputs in the console and it minimizes to the notification area
Reproduce
Steps to reproduce the behavior:
firejail qbittorrent
Environment
Debian 10 stable with KDE
Firejail 0.9.62
Additional context
The notification icon shows when running it without firejail
Checklist
https://github.com/netblue30/firejail/issues/1139
)--profile=PROFILENAME
is used to set the right profile.The text was updated successfully, but these errors were encountered: