Skip to content

TaskManager: Restrict parents process to those sharing the same cgroup

Méven Car requested to merge work/meven/plasma-desktop-bug-417457 into master

Context: https://phabricator.kde.org/D25874

Tested playing video in:

  • vlc, opening a video file in dolphin (the bug)
  • firefox playing youtube
  • chromium playing youtube

BUG: 417457

The bug was due to a race condition : the current stream pids are not immediately available in function streamsForPid(pid) { (https://phabricator.kde.org/D25874), which means any tasks will treat their parent process as their stream owner for a brief moment, and this is then cached.

The fix prevents this to happen ever again, leveraging the KIO cgroup/systemd startup per-app scope.

cc @broulik @ngraham

Replaces !407 (closed) !418 (closed)

Edited by Méven Car

Merge request reports