]> git.ipfire.org Git - thirdparty/libcgroup.git/commit
Fixed cg_chmod_recursive
authorJan Safranek <jsafrane@redhat.com>
Wed, 1 Jun 2011 09:10:51 +0000 (11:10 +0200)
committerJan Safranek <jsafrane@redhat.com>
Fri, 3 Jun 2011 08:27:47 +0000 (10:27 +0200)
commitea3c8d98570d062f288c9ce145977b648115aa10
tree4f559316572483fa75487b376086708474977d9a
parent46b1a9232271496a5595e08cef66ce4834f3afb6
Fixed cg_chmod_recursive

The function changes permissions only in the first hierarchy, but it should
change it in all of them.

Reproducer:
1) have cpuacct and freezer mounted separately, i.e.:
$ lssubsys -m
cpuacct /sys/fs/cgroup/cpuacct
freezer /sys/fs/cgroup/freezer

2) create a group with specific permissions:
$ cgcreate -f 700  -g freezer,cpuacct:/test

Result:
$ ls -la /sys/fs/cgroup/freezer/test/
-rwx------. 1 jsafrane jsafrane 0 May 31 09:16 cgroup.clone_children
-rwx------. 1 jsafrane jsafrane 0 May 31 09:16 cgroup.event_control
(-> first controller is fine)

$ ls -la /sys/fs/cgroup/cpuacct/test/
-rw-r--r--. 1 jsafrane jsafrane 0 May 31 09:16 cgroup.clone_children
--w--w--w-. 1 jsafrane jsafrane 0 May 31 09:16 cgroup.event_control
(-> second controller is wrong, it should be -rwx------)

Changelog:
 - v2: fixed return code of cg_chmod_recursive_controller when fts_read fails
 - v3: fixed error code when malloc fails

Signed-off-by: Jan Safranek <jsafrane@redhat.com>
Acked-By: Ivana Hutarova Varekova<varekova@redhat.com>
src/api.c