> On 2010-11-10 17:30:41, Chani Armitage wrote: > > no, I don't think this is correct. > > > > check out line 880: > > ImmutabilityType Applet::immutability() const > > that function doesn't return d->immutability, but the strongest of that and > > various other checks. and it suggests that Corona::immutability() should be > > responsible for checking kiosk things. > > > > I'm not sure what checkImmutability is supposed to be doing, but setting > > d->immutability isn't it ;) > > > > ...also note that, once you set SystemImmutable, you can't unset it. at > > least that's how the code read to me. so test that kiosk can both lock > > *and* unlock things ;)
i think both are ssafe, i.e if the immutability isn't set, the proper (aka more restrictive) one should always be returned. setting it also at applet level is like to be uhm more sure about it. systemimmutable wasn't supposed to be changed at runtime in any way, and after the patch still isn't thinking about it, a problem of the patch could be that now the systemimmutable ness would be written into the config file, so wouldn't be possible to unlock everything by just changing from kdeglobals (with kde not running) but the change would have to be propagated by hand into plasma-desktop-appletsrc - Marco ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://svn.reviewboard.kde.org/r/5823/#review8624 ----------------------------------------------------------- On 2010-11-10 16:39:31, Lubos Lunak wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://svn.reviewboard.kde.org/r/5823/ > ----------------------------------------------------------- > > (Updated 2010-11-10 16:39:31) > > > Review request for Plasma and Marco Martin. > > > Summary > ------- > > Probably as a result of r800724, Plasma doesn't seem to follow Kiosk > restrictions at all (just grep the sources for where SystemImmutable is > assigned to something - it isn't). This patch may possibly be a suitable fix. > > > Diffs > ----- > > trunk/KDE/kdelibs/plasma/applet.cpp 1182745 > > Diff: http://svn.reviewboard.kde.org/r/5823/diff > > > Testing > ------- > > > Thanks, > > Lubos > >
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
