Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Firejail breaks media controls in Plasma taskbar #1619

Closed
rc2dev opened this issue Oct 26, 2017 · 8 comments
Closed

Firejail breaks media controls in Plasma taskbar #1619

rc2dev opened this issue Oct 26, 2017 · 8 comments
Labels
information_old (Deprecated; use "doc-todo" or "needinfo" instead) Information was/is required

Comments

@rc2dev
Copy link
Contributor

rc2dev commented Oct 26, 2017

I posted this bug to KDE.

Apparently, the cause is that _NET_WM_PID is different from the PID of the jailed program.

$ pidof vlc
3466

$ pidof firejail
3457

$ qdbus org.freedesktop.DBus / org.freedesktop.DBus.GetConnectionUnixProcessID \
  org.mpris.MediaPlayer2.vlc
3466

$ xprop | grep PID
_NET_WM_PID(CARDINAL) = 9
@netblue30 netblue30 added the information_old (Deprecated; use "doc-todo" or "needinfo" instead) Information was/is required label Nov 1, 2017
@smitsohu
Copy link
Collaborator

smitsohu commented Nov 1, 2017

just tried it out in Mate, it is also the reason for #258. Mate labels the titlebar with "as superuser" if _NET_WM_PID indicates root ownership.

@smitsohu
Copy link
Collaborator

smitsohu commented Nov 2, 2017

Apparently, the cause is that _NET_WM_PID is different from the PID of the jailed program.

The reason is that _NET_WM_PID is set to the PID inside the sandbox. Whatever sets _NET_WM_PID, it is not aware of PID namespaces.

@smitsohu
Copy link
Collaborator

smitsohu commented Nov 3, 2017

See also #1084

@smitsohu
Copy link
Collaborator

smitsohu commented Nov 4, 2017

It looks Gnome have chosen a workaround for Flatpak only: https://bugzilla.gnome.org/show_bug.cgi?id=772615

@chiraag-nataraj
Copy link
Collaborator

Is this still an issue? I presume it is...

@rc2dev
Copy link
Contributor Author

rc2dev commented May 23, 2019

@chiraag-nataraj I'm not using Plasma currently, so I can't say.

@chiraag-nataraj
Copy link
Collaborator

Okay, I'll close this for now. Please feel free to re-open if you're experiencing this and we'll take a look then.

@Vincent43
Copy link
Collaborator

This is fixed in KDE Frameworks 5.61

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
information_old (Deprecated; use "doc-todo" or "needinfo" instead) Information was/is required
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
5 participants
@rc2dev @netblue30 @chiraag-nataraj @smitsohu @Vincent43