topimiettinen (Topi Miettinen)
User

Projects

User does not belong to any projects.

Today

  • Clear sailing ahead.

Tomorrow

  • Clear sailing ahead.

Saturday

  • Clear sailing ahead.

User Details

User Since
Apr 15 2019, 6:53 PM (5 w, 2 d)
Availability
Available

Recent Activity

Mon, May 20

topimiettinen requested review of D21310: kcm_systemd: update and sort capabilities.
Mon, May 20, 6:02 PM
topimiettinen committed R106:446a3b368813: ksysguardd: show Mandatory Access Control (SELinux/AppArmor) context (authored by topimiettinen).
ksysguardd: show Mandatory Access Control (SELinux/AppArmor) context
Mon, May 20, 5:14 PM
topimiettinen closed D20853: ksysguardd: show Mandatory Access Control (SELinux/AppArmor) context.
Mon, May 20, 5:14 PM · Plasma
topimiettinen committed R111:e3894f9408cc: libksysguard: show Mandatory Access Control (SELinux/AppArmor) context (authored by topimiettinen).
libksysguard: show Mandatory Access Control (SELinux/AppArmor) context
Mon, May 20, 5:12 PM
topimiettinen closed D20854: libksysguard: show Mandatory Access Control (SELinux/AppArmor) context.
Mon, May 20, 5:12 PM · Plasma
topimiettinen updated the summary of D20854: libksysguard: show Mandatory Access Control (SELinux/AppArmor) context.
Mon, May 20, 5:06 PM · Plasma
topimiettinen updated the diff for D20854: libksysguard: show Mandatory Access Control (SELinux/AppArmor) context.

update to reflect that this also shows AppArmor context

Mon, May 20, 5:04 PM · Plasma

Sat, May 18

topimiettinen updated the summary of D20853: ksysguardd: show Mandatory Access Control (SELinux/AppArmor) context.
Sat, May 18, 1:40 PM · Plasma
topimiettinen updated the diff for D20853: ksysguardd: show Mandatory Access Control (SELinux/AppArmor) context.

Updated to reflect that this is also useful for AppArmor

Sat, May 18, 1:38 PM · Plasma
topimiettinen committed R111:3eb5e842ae64: process: add support for cgroups (authored by topimiettinen).
process: add support for cgroups
Sat, May 18, 9:09 AM
topimiettinen closed D20787: process: add support for cgroups.
Sat, May 18, 9:08 AM · Plasma
topimiettinen committed R106:11af9d11a457: ProcessList: add support for cgroups (authored by topimiettinen).
ProcessList: add support for cgroups
Sat, May 18, 9:03 AM
topimiettinen closed D20786: ProcessList: add support for cgroups.
Sat, May 18, 9:03 AM · Plasma
topimiettinen committed R106:892c895bda83: ProcessList: add NoNewPrivileges (authored by topimiettinen).
ProcessList: add NoNewPrivileges
Sat, May 18, 8:56 AM
topimiettinen closed D20584: ProcessList: add NoNewPrivileges.
Sat, May 18, 8:56 AM · Plasma
topimiettinen added a comment to D20853: ksysguardd: show Mandatory Access Control (SELinux/AppArmor) context.

Like D20854, this should be also applicable to AppArmor.

Sat, May 18, 8:05 AM · Plasma
topimiettinen added a comment to D20854: libksysguard: show Mandatory Access Control (SELinux/AppArmor) context.

I noticed that when the system uses AppArmor instead of SELinux, this also happens to show AppArmor profile and status. Then at least user facing bits should not be so SELinux oriented to avoid confusion. I suppose AppArmor is more widely used than SELinux.

Sat, May 18, 8:03 AM · Plasma

Mon, Apr 29

topimiettinen updated subscribers of D20901: Hide the new NNP column by default.

requested by @broulik

Mon, Apr 29, 3:40 PM · Plasma
topimiettinen requested review of D20901: Hide the new NNP column by default.
Mon, Apr 29, 3:36 PM · Plasma
topimiettinen updated the diff for D20854: libksysguard: show Mandatory Access Control (SELinux/AppArmor) context.

hide selinux column view by default

Mon, Apr 29, 3:35 PM · Plasma
topimiettinen updated the diff for D20787: process: add support for cgroups.

hide cgroup column by default

Mon, Apr 29, 3:35 PM · Plasma

Sun, Apr 28

topimiettinen added a comment to D20853: ksysguardd: show Mandatory Access Control (SELinux/AppArmor) context.

Out of curiosity, do you use the remote processlist?

Sun, Apr 28, 6:48 PM · Plasma

Sat, Apr 27

topimiettinen added a comment to D20853: ksysguardd: show Mandatory Access Control (SELinux/AppArmor) context.

Example output:

Sat, Apr 27, 12:16 PM · Plasma
topimiettinen updated the diff for D20853: ksysguardd: show Mandatory Access Control (SELinux/AppArmor) context.

avoid memory allocations and reuse existing validation function

Sat, Apr 27, 12:13 PM · Plasma
topimiettinen updated the diff for D20786: ProcessList: add support for cgroups.

avoid memory allocations and reuse existing validation function

Sat, Apr 27, 12:11 PM · Plasma
topimiettinen added inline comments to D20786: ProcessList: add support for cgroups.
Sat, Apr 27, 10:06 AM · Plasma
topimiettinen updated the diff for D20786: ProcessList: add support for cgroups.

more trimming logic fixes

Sat, Apr 27, 10:03 AM · Plasma
topimiettinen updated the diff for D20786: ProcessList: add support for cgroups.

improve trimming, thanks to @anthonyfieroni

Sat, Apr 27, 8:33 AM · Plasma
topimiettinen updated the diff for D20786: ProcessList: add support for cgroups.

free strdup'ed string

Sat, Apr 27, 8:26 AM · Plasma
topimiettinen updated the diff for D20853: ksysguardd: show Mandatory Access Control (SELinux/AppArmor) context.

free strdup'ed string

Sat, Apr 27, 8:22 AM · Plasma

Fri, Apr 26

topimiettinen added a comment to D20854: libksysguard: show Mandatory Access Control (SELinux/AppArmor) context.

Fedora with SELinux

Fri, Apr 26, 9:57 PM · Plasma
topimiettinen requested review of D20854: libksysguard: show Mandatory Access Control (SELinux/AppArmor) context.
Fri, Apr 26, 9:56 PM · Plasma
topimiettinen requested review of D20853: ksysguardd: show Mandatory Access Control (SELinux/AppArmor) context.
Fri, Apr 26, 9:53 PM · Plasma
topimiettinen updated the diff for D20786: ProcessList: add support for cgroups.

trim trailing newlines

Fri, Apr 26, 9:47 PM · Plasma
topimiettinen updated the diff for D20787: process: add support for cgroups.

fix copy-pasta

Fri, Apr 26, 9:43 PM · Plasma
topimiettinen added a comment to D20787: process: add support for cgroups.

Looks much better!

Fri, Apr 26, 8:22 PM · Plasma
topimiettinen updated the diff for D20787: process: add support for cgroups.

phabricate only one git commit, sorry

Fri, Apr 26, 8:17 PM · Plasma
topimiettinen updated the diff for D20787: process: add support for cgroups.

Trim trailing newline for proper vertical alignment

Fri, Apr 26, 8:15 PM · Plasma

Thu, Apr 25

topimiettinen added a comment to D20787: process: add support for cgroups.

It looks like the CGroup rows are visually not vertically aligned to center like other cells, but instead they are aligned to the top. However, I'd expect that the default item of switch in ProcessModel.cpp:1693 should implement this, like it does for e.g. HeadingName. So I'm a bit puzzled why this happens.

Thu, Apr 25, 1:53 PM · Plasma

Wed, Apr 24

topimiettinen abandoned D20583: process: add NoNewPrivileges.

Abandoned in favour of D20645.

Wed, Apr 24, 5:09 PM · Plasma
topimiettinen updated the diff for D20787: process: add support for cgroups.

update per comments from @davidedmundson and @apol

Wed, Apr 24, 4:49 PM · Plasma
topimiettinen added a comment to D20645: process: add NoNewPrivileges.

Do you have KDE commit access?

Wed, Apr 24, 4:32 PM · Plasma
topimiettinen added a comment to D20787: process: add support for cgroups.

Wed, Apr 24, 1:06 PM · Plasma
topimiettinen requested review of D20787: process: add support for cgroups.
Wed, Apr 24, 1:03 PM · Plasma
topimiettinen added a comment to D20786: ProcessList: add support for cgroups.

Sample output:

Wed, Apr 24, 1:00 PM · Plasma
topimiettinen requested review of D20786: ProcessList: add support for cgroups.
Wed, Apr 24, 12:57 PM · Plasma
topimiettinen updated the diff for D20645: process: add NoNewPrivileges.

update also remote protocol

Wed, Apr 24, 7:29 AM · Plasma

Apr 18 2019

topimiettinen updated the diff for D20645: process: add NoNewPrivileges.

Updated per comments from @davidedmundson.

Apr 18 2019, 5:34 PM · Plasma

Apr 17 2019

topimiettinen added a comment to D20583: process: add NoNewPrivileges.

It's not static, the thread can enable the mode with prctl(PR_SET_NO_NEW_PRIVS). I suppose Process::Status could be used? Maybe I made a mistake, now there is D20645.

Apr 17 2019, 7:43 PM · Plasma
topimiettinen requested review of D20645: process: add NoNewPrivileges.
Apr 17 2019, 7:39 PM · Plasma

Apr 15 2019

topimiettinen added a comment to D20584: ProcessList: add NoNewPrivileges.

This is related to D20583.

Apr 15 2019, 7:28 PM · Plasma
topimiettinen requested review of D20584: ProcessList: add NoNewPrivileges.
Apr 15 2019, 7:23 PM · Plasma
topimiettinen added a comment to D20583: process: add NoNewPrivileges.

Apr 15 2019, 7:13 PM · Plasma
topimiettinen requested review of D20583: process: add NoNewPrivileges.
Apr 15 2019, 7:04 PM · Plasma