cgroup: fix exit() vs rmdir() race
authorLi Zefan <lizefan@huawei.com>
Thu, 24 Jan 2013 06:43:28 +0000 (14:43 +0800)
committerTejun Heo <tj@kernel.org>
Mon, 18 Feb 2013 17:08:10 +0000 (09:08 -0800)
commit71b5707e119653039e6e95213f00479668c79b75
treecb0dfe84c3b8ec46efce2aa4185342a3a75cc3a1
parent9ed8a659703876a9fe96ab86d1b296c2f0084242
cgroup: fix exit() vs rmdir() race

In cgroup_exit() put_css_set_taskexit() is called without any lock,
which might lead to accessing a freed cgroup:

thread1                           thread2
---------------------------------------------
exit()
  cgroup_exit()
    put_css_set_taskexit()
      atomic_dec(cgrp->count);
                                   rmdir();
      /* not safe !! */
      check_for_release(cgrp);

rcu_read_lock() can be used to make sure the cgroup is alive.

Signed-off-by: Li Zefan <lizefan@huawei.com>
Signed-off-by: Tejun Heo <tj@kernel.org>
Cc: stable@vger.kernel.org
kernel/cgroup.c