Review Request: Load KDE Resource Restrictions from Kiosk profiles

Petr Mrázek xmraze03 at stud.fit.vutbr.cz
Thu May 20 23:18:13 BST 2010


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.kde.org/r/4059/
-----------------------------------------------------------

(Updated 2010-05-20 22:18:12.749569)


Review request for kdelibs.


Changes
-------

Seems like I put in the wrong patch at first. Oh well.

This one has none of the KAuth changes.


Summary
-------

Kiosk restrictions on resources couldn't be specified in the Kiosk profiles before. This should fix that.
I also cleaned up some other related bits and moved the check for resource restrictions into KAuthorized.


Diffs (updated)
-----

  /trunk/KDE/kdelibs/kdecore/config/kconfig.cpp 1125109 
  /trunk/KDE/kdelibs/kdecore/kernel/kauthorized.h 1125109 
  /trunk/KDE/kdelibs/kdecore/kernel/kauthorized.cpp 1125109 
  /trunk/KDE/kdelibs/kdecore/kernel/kcomponentdata.cpp 1125109 
  /trunk/KDE/kdelibs/kdecore/kernel/kstandarddirs.h 1125109 
  /trunk/KDE/kdelibs/kdecore/kernel/kstandarddirs.cpp 1125109 

Diff: http://reviewboard.kde.org/r/4059/diff


Testing
-------

I've made a user profile, assigned it to a dummy user and restricted the wallpaper resource in that profile's kdeglobals config file. It works as expected -- user owned wallpapers aren't listed in desktop activity settings.
Same kind of test done with the 'config' resource reveals other problems. For example plasma-desktop does indeed respect the restriction,
but KDE's color settings seem to be still loaded from the user's configuration files.


Thanks,

Petr





More information about the kde-core-devel mailing list