Bug 438626 - Killing malfunctions in tree mode
Summary: Killing malfunctions in tree mode
Status: RESOLVED FIXED
Alias: None
Product: plasma-systemmonitor
Classification: Applications
Component: general (show other bugs)
Version: 5.22.0
Platform: openSUSE Linux
: HI major
Target Milestone: ---
Assignee: KSysGuard Developers
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2021-06-14 15:57 UTC by Alois Wohlschlager
Modified: 2021-06-28 12:04 UTC (History)
3 users (show)

See Also:
Latest Commit:
Version Fixed In: 5.22.3


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Alois Wohlschlager 2021-06-14 15:57:44 UTC
SUMMARY

When invoking the "end process" function in tree mode, the system monitor wants to kill a different process from the selected one.

STEPS TO REPRODUCE
1. Open system monitor (process page)
2. Ensure tree mode is active
3. Select a process and try to kill it (with Del key, context menu or End Process button)

OBSERVED RESULT

System monitor displays a dialog which shows it wants to kill a different process (usually systemd or kthreadd). When confirming this window, it actually attempts to send SIGTERM to the wrong process (which obviously fails).

Sometimes, it does not display a process name, and displays "PID %1, User %2". No SIGTERM is sent in that case. I believe this is a symptom of the same underlying issue.

EXPECTED RESULT

System monitor displays the correct process in the confirmation dialog, and SIGTERM goes to the intended process.

SOFTWARE/OS VERSIONS
Windows: 
macOS: 
Linux/KDE Plasma: 
(available in About System)
KDE Plasma Version: 5.22.0
KDE Frameworks Version: 5.82.0
Qt Version: 5.15.2

ADDITIONAL INFORMATION

The kill functionality seems to work fine in list mode. The bug also happens on git master.
Comment 1 Nate Graham 2021-06-15 20:47:57 UTC
Can confirm.
Comment 2 Bug Janitor Service 2021-06-23 07:16:51 UTC
A possibly relevant merge request was started @ https://invent.kde.org/plasma/plasma-systemmonitor/-/merge_requests/148
Comment 3 David Redondo 2021-06-28 12:04:22 UTC
Git commit 1727ac0d632deeed6278819005aab6abc11fc088 by David Redondo.
Committed on 28/06/2021 at 12:04.
Pushed by davidre into branch 'master'.

Pass parent index when fetching selection data
FIXED-IN:5.22.3

M  +3    -3    src/faces/processtable/contents/ui/ProcessTableView.qml

https://invent.kde.org/plasma/plasma-systemmonitor/commit/1727ac0d632deeed6278819005aab6abc11fc088
Comment 4 David Redondo 2021-06-28 12:04:47 UTC
Git commit 8cc770f9819f83d0dc60d8eeb923698f82be011d by David Redondo.
Committed on 28/06/2021 at 12:04.
Pushed by davidre into branch 'Plasma/5.22'.

Pass parent index when fetching selection data
FIXED-IN:5.22.3


(cherry picked from commit 1727ac0d632deeed6278819005aab6abc11fc088)

M  +3    -3    src/faces/processtable/contents/ui/ProcessTableView.qml

https://invent.kde.org/plasma/plasma-systemmonitor/commit/8cc770f9819f83d0dc60d8eeb923698f82be011d