----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126203/#review88953 -----------------------------------------------------------
greeter/main.cpp (line 32) <https://git.reviewboard.kde.org/r/126203/#comment60901> Both `prctl()` and `PR_SET_DUMPABLE` seem to be defined in `sys/prctl.h` on Linux, so this include should not be necessary. kcheckpass/kcheckpass.c (line 61) <https://git.reviewboard.kde.org/r/126203/#comment60902> Ditto here. - Raphael Kubo da Costa On Nov. 30, 2015, 10:01 a.m., Martin Gräßlin wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/126203/ > ----------------------------------------------------------- > > (Updated Nov. 30, 2015, 10:01 a.m.) > > > Review request for Plasma and Tobias Berner. > > > Repository: kscreenlocker > > > Description > ------- > > Setting the PR_SET_DUMPABLE flag to 0 for the security relevant > command kcheckpass and kscreenlocker_greet. If one wants to gdb into > the running command it will result in: > > ptrace: Operation not permitted. > > For kscreenlocker_greet ptrace is permitted in testing mode. > > As root it's still possible to attach to the process. > > --- > > @Tobias: I assume this is a strong linux-ism. Is there a FreeBSD compareable > functionality? > > I'm considering to push this explicitly without an ifdef. It's a new security > feature and I want to make non-Linux systems aware of the fact that it adds a > new feature and that a replacement should be added. > > > Diffs > ----- > > greeter/main.cpp e4e679e7ef40b319665428281fdba5f4e0b4eb25 > kcheckpass/kcheckpass.c fd2d2215bf2199f159a121bb0ce08e7b2b254aaa > > Diff: https://git.reviewboard.kde.org/r/126203/diff/ > > > Testing > ------- > > Tried to gdb into the processes: failed > Tried to gdb into kscreenlocker_greet --testing: succeeded > Tried to gdb into kscreenlocker_greet as root: succeeded > > > Thanks, > > Martin Gräßlin > >
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel