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)
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

No differences found